Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AJ-1277] add v1 deleteRecords API to control plane preview swagger page #935

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

mspector
Copy link
Contributor

Reminder:

Releasing WDS

PRs merged into main will not automatically generate a PR in https://github.com/broadinstitute/terra-helmfile to update the WDS image deployed to kubernetes - this action will need to be triggered manually by running the following github action: https://github.com/DataBiosphere/terra-workspace-data-service/actions/workflows/tag-and-publish.yml. Dont forget to provide a Jira Id when triggering the manual action, if no Jira ID is provided the action will not fully succeed.

After you manually trigger the github action (and it completes with no errors), you must go to the terra-helmfile repo and verify that this generated a PR that merged successfully.

The terra-helmfile PR merge will then generate a PR in leonardo. This will automerge if all tests pass, but if jenkins tests fail it will not; be sure to watch it to ensure it merges. To trigger jenkins retest simply comment on PR with "jenkins retest".

Keeping Docs Up To Date

If you make changes to the github actions or workflows, particularly when they are run or which other workflows they call, please update the GHA wiki page to stay up to date.

@mspector mspector marked this pull request as ready for review September 20, 2024 17:36
Copy link

sonarcloud bot commented Sep 20, 2024

@mspector mspector merged commit 2501818 into main Sep 20, 2024
25 checks passed
@mspector mspector deleted the AJ-1277_delete-records-v1-preview-swagger branch September 20, 2024 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants