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.

Sign up for GitHub

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

Jump to bottom

Cromwell 34 updates for Aug 7-9ish release #151

Closed
jmthibault79 opened this issue Aug 7, 2018 · 3 comments · Fixed by #152
Closed

Cromwell 34 updates for Aug 7-9ish release #151

jmthibault79 opened this issue Aug 7, 2018 · 3 comments · Fixed by #152
Assignees
@jmthibault79 @ruchim

Comments

@jmthibault79
Copy link

jmthibault79 commented Aug 7, 2018 •
edited
Loading

https://github.com/broadinstitute/firecloud-develop/pull/1293
@jmthibault79 jmthibault79 self-assigned this Aug 7, 2018
@jmthibault79
Copy link
Author

jmthibault79 commented Aug 7, 2018 •
edited
Loading

Proposed FC testing plan, based on PR:

  • Put it on a FIAB "A" and run a smoke test
  • Observe that /stats returns 403 on FIAB A
  • Ruchi tests validation changes on FIAB A by running the specific workflow 100x
    ** Run the same test on a dev FIAB "B" without these changes to see that the problem does exist there
  • Ruchi tests that stdout/stderr does update on FIAB A and does not on FIAB B
  • Doug tests Martha scopes on FIAB A or after merging to dev, as appropriate
  • Gary runs the alpha-perf test after merging to alpha

@davidangb
Copy link
Contributor

👍 for test plan, assuming we (Ruchi) has appropriate access to monitor CPU on FIAB A

@jmthibault79
Copy link
Author

Cromwell version hotfix PRs:
https://github.com/broadinstitute/firecloud-develop/pull/1299
broadinstitute/agora#263
broadinstitute/rawls#972

Cromwell config changes:
https://github.com/broadinstitute/firecloud-develop/pull/1296

@jmthibault79 jmthibault79 changed the title Cromwell 34 updates for Aug 8 release Cromwell 34 updates for Aug 7-9ish release Aug 8, 2018
@davidangb davidangb mentioned this issue Aug 9, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmthibault79 jmthibault79

@ruchim ruchim

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

RC_Yong_Kang_Grey draft relnotes DataBiosphere/firecloud-app
3 participants
@jmthibault79 @davidangb @ruchim