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

dev --> staging #1384

Merged
merged 2 commits into from
Oct 9, 2024
Merged

dev --> staging #1384

merged 2 commits into from
Oct 9, 2024

Conversation

ekiernan
Copy link
Contributor

@ekiernan ekiernan commented Oct 9, 2024

dev --> staging

kevinpalis and others added 2 commits October 3, 2024 15:14
* Making sure temporary credentials or keys don't get pushed by accident

* Adding a GHA that can authenticate using a service account in GCP

* Plugging in the magical auth system to finally connect to Terra

* Adding quotes in vars. Disabling PR trigger to avoid duplicated runs for now.

* Fixing newline. Adding debug lines

* Debugging why the access token is not being accepted

* Adding time to token validity. Maybe it expired before getting used

* Removing printings

* retrigger

* Retrigger

* Trying out a different token

* Setting back to access token

* try setting scopes directly

* Removing debug flag for cleaner output.

* Adding getting submission list

* Changing to warp tests

* Switching back

* Adding Firecloud API script

* Trying out new py class

* Changing variables to get this pipeline to run automatically

* Plugging in actual values to test this. Increasing polling intervals

* Fixing path mistake

* Commenting out test code. Fixing error with memoryRetryMultiplier

* Returning workflowID

* Setting to unbuffered so we see printings from py

* Adding functionality to handle multiple workflows in a single run.

* Trying to parse the monitorSubmission properly.

* Trying to parse the monitorSubmission properly.

* Trying to parse the monitorSubmission properly.

* Trying to parse the monitorSubmission properly.

* It looks like python printings messes up bash inputs. Cleaning it up.

* Creating a map of workflows and their statuses.

* Creating a map of workflows and their statuses.

* Adding summary printings

* Parameterizing call_cache

* Adding a summary in the event of a failure.

* Better summary printing. Covering both success and failure.

* Removing stderr redirection to see where things are failing

* Finally found what was wrong. Token was expiring before finishing the polling step. Setting it back to the default, 1 hour.

* Parsing the outputs dictionary properly.

* One more try with the parsing of outputs. Adding an id to pipeline run step

* Handling int to str in json

* Better summary formatting

* Better summary formatting

* Some icons.

* Adding note to make sure folks don't change the magic string

* Adding proper triggers

* Removing test trigger

---------

Co-authored-by: Jack Warren <[email protected]>
added atac_nhash_id to multiome sci test json
Copy link

github-actions bot commented Oct 9, 2024

DO NOT SQUASH MERGE!!!

Copy link

github-actions bot commented Oct 9, 2024

🔍Changelog Validation Results:

basename: missing operand
Try 'basename --help' for more information.
Comparing changelogs for pipelines that differ from the versions on 'origin/staging':
All changelog files are valid for this release.

Copy link

github-actions bot commented Oct 9, 2024

🔍Version Validation Results:

basename: missing operand
Try 'basename --help' for more information.
Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@ekiernan ekiernan merged commit 3ff3c74 into staging Oct 9, 2024
9 of 10 checks passed
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