larryh/corso
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Prevent the static purge scripts running in combination without a user (#2832)

Browse Source 
---

#### Does this PR need a docs update or release note?

- [ ] ✅ Yes, it's included
- [ ] 🕐 Yes, but in a later PR
- [x] ⛔ No

#### Type of change

<!--- Please check the type of change your PR introduces: --->
- [ ] 🌻 Feature
- [x] 🐛 Bugfix
- [ ] 🗺️ Documentation
- [ ] 🤖 Test
- [ ] 💻 CI/Deployment
- [ ] 🧹 Tech Debt/Cleanup

#### Issue(s)

<!-- Can reference multiple issues. Use one of the following "magic words" - "closes, fixes" to auto-close the Github issue. -->
* #<issue>

#### Test Plan

<!-- How will this be tested prior to merging.-->
- [x] 💪 Manual
- [ ] ⚡ Unit test
- [ ] 💚 E2E
This commit is contained in:
Georgi Matev 2023-03-16 20:07:34 -04:00 committed by GitHub
parent 26bf24c77e
commit bf2f66398b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
.github/actions/purge-m365-user-data/action.yml vendored
View File

@ -27,6 +27,10 @@ inputs:
description: Secret value of for AZURE_CLIENT_SECRET description: Secret value of for AZURE_CLIENT_SECRET
azure-tenant-id: azure-tenant-id:
description: Secret value of for AZURE_TENANT_ID description: Secret value of for AZURE_TENANT_ID
m365-admin-user:
description: Secret value of for M365TENANT_ADMIN_USER
m365-admin-password:
description: Secret value of for M365TENANT_ADMIN_PASSWORD
runs: runs:
using: composite using: composite
@ -54,7 +58,7 @@ runs:
./setRetention.ps1 ./setRetention.ps1
- name: Run the static purge script set - name: Run the static purge script set
if: ${{ inputs.folder-prefix == '' }} if: ${{ inputs.folder-prefix == '' && inputs.user != '' }}
shell: pwsh shell: pwsh
working-directory: ./src/cmd/purge/scripts working-directory: ./src/cmd/purge/scripts
env: env: