Skip to content

Update BRH staging to use WORKFLOW task tokens#1838

Merged
nss10 merged 5 commits into
masterfrom
chore/update_fence_brhstaging
Jul 15, 2026
Merged

Update BRH staging to use WORKFLOW task tokens#1838
nss10 merged 5 commits into
masterfrom
chore/update_fence_brhstaging

Conversation

@nss10

@nss10 nss10 commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Link to Jira ticket if there is one:

Environments

  • BRH Staging

Description of changes

  • Add WORKFLOW task token type in BRH staging

@nss10
nss10 marked this pull request as ready for review July 14, 2026 14:28
@nss10
nss10 requested a review from briennal as a code owner July 14, 2026 14:28
Avantol13
Avantol13 previously approved these changes Jul 14, 2026
@github-actions

Copy link
Copy Markdown

Integration Tests

Test summary after running integration tests

filepath passed failed error skipped SUBTOTAL
tests/test_oauth2.py 0 0 0 15 15
tests/test_dicom_viewer.py 0 0 0 4 4
tests/test_pfb_export.py 0 0 0 1 1
tests/test_guppy_service.py 0 0 0 13 13
tests/test_gen3_workflow.py 0 0 28 1 29
tests/test_data_upload.py 0 0 0 9 9
tests/test_audit_service.py 0 3 0 3 6
tests/test_ras_authn.py 0 0 0 3 3
tests/test_ras_passport.py 0 0 0 3 3
tests/test_user_data_library.py 0 0 0 4 4
tests/test_study_viewer.py 0 0 0 3 3
tests/test_user_login_activation.py 2 0 0 1 3
tests/test_discoverypage.py 0 0 0 1 1
tests/test_gen3ff_landing_page.py 0 0 0 2 2
tests/test_register_user.py 0 0 0 2 2
tests/test_graph_submit_and_query.py 12 1 0 1 14
tests/test_aggregate_mds.py 0 0 0 1 1
tests/test_google_data_access.py 0 0 0 1 1
tests/test_env_sanity.py 0 0 0 1 1
tests/test_etl.py 0 0 0 1 1
tests/test_gen3_sdk.py 0 0 0 1 1
tests/test_gen3client.py 0 0 0 1 1
tests/test_dbgap.py 4 0 0 1 5
tests/test_presigned_url.py 7 0 0 0 7
tests/test_centralized_auth.py 16 0 0 0 16
tests/test_requestor.py 6 0 0 0 6
tests/test_drs_endpoint.py 6 0 0 0 6
tests/test_user_token.py 5 0 0 0 5
tests/test_login_page.py 2 0 0 0 2
tests/test_oidc_client.py 2 0 0 0 2
tests/test_homepage.py 1 0 0 0 1
tests/test_client_credentials.py 1 0 0 0 1
tests/test_fence_admin.py 2 0 0 0 2
tests/test_workspace.py 0 1 0 0 1
tests/test_manifest_service.py 0 1 0 0 1
TOTAL 66 6 28 73 173

Test summary after rerunning failed integration tests

filepath passed failed error SUBTOTAL
tests/test_gen3_workflow.py 0 0 28 28
tests/test_workspace.py 0 1 0 1
tests/test_manifest_service.py 0 1 0 1
tests/test_audit_service.py 0 3 0 3
tests/test_graph_submit_and_query.py 1 0 0 1
TOTAL 1 5 28 34

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

@github-actions

Copy link
Copy Markdown

Integration Tests

Test summary after running integration tests

filepath passed failed skipped SUBTOTAL
tests/test_oauth2.py 0 0 15 15
tests/test_dicom_viewer.py 0 0 4 4
tests/test_pfb_export.py 0 0 1 1
tests/test_guppy_service.py 0 0 13 13
tests/test_audit_service.py 0 3 3 6
tests/test_ras_authn.py 0 0 3 3
tests/test_ras_passport.py 0 0 3 3
tests/test_graph_submit_and_query.py 12 1 1 14
tests/test_gen3_workflow.py 26 2 1 29
tests/test_dbgap.py 4 0 1 5
tests/test_user_data_library.py 0 0 4 4
tests/test_study_viewer.py 0 0 3 3
tests/test_data_upload.py 0 0 9 9
tests/test_discoverypage.py 0 0 1 1
tests/test_user_login_activation.py 2 0 1 3
tests/test_gen3ff_landing_page.py 0 0 2 2
tests/test_register_user.py 0 0 2 2
tests/test_aggregate_mds.py 0 0 1 1
tests/test_google_data_access.py 0 0 1 1
tests/test_env_sanity.py 0 0 1 1
tests/test_gen3_sdk.py 0 0 1 1
tests/test_gen3client.py 0 0 1 1
tests/test_etl.py 0 0 1 1
tests/test_centralized_auth.py 16 0 0 16
tests/test_presigned_url.py 7 0 0 7
tests/test_requestor.py 6 0 0 6
tests/test_drs_endpoint.py 6 0 0 6
tests/test_user_token.py 5 0 0 5
tests/test_login_page.py 2 0 0 2
tests/test_oidc_client.py 2 0 0 2
tests/test_homepage.py 1 0 0 1
tests/test_client_credentials.py 1 0 0 1
tests/test_fence_admin.py 2 0 0 2
tests/test_workspace.py 0 1 0 1
tests/test_manifest_service.py 0 1 0 1
TOTAL 92 8 73 173

Test summary after rerunning failed integration tests

filepath passed failed SUBTOTAL
tests/test_graph_submit_and_query.py 1 0 1
tests/test_gen3_workflow.py 1 1 2
tests/test_workspace.py 0 1 1
tests/test_audit_service.py 0 3 3
tests/test_manifest_service.py 0 1 1
TOTAL 2 6 8

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

@github-actions

Copy link
Copy Markdown

Integration Tests

Test summary after running integration tests

filepath passed failed skipped SUBTOTAL
tests/test_dicom_viewer.py 0 0 4 4
tests/test_oauth2.py 0 0 15 15
tests/test_pfb_export.py 0 0 1 1
tests/test_guppy_service.py 0 0 13 13
tests/test_audit_service.py 0 3 3 6
tests/test_ras_authn.py 0 0 3 3
tests/test_ras_passport.py 0 0 3 3
tests/test_graph_submit_and_query.py 12 1 1 14
tests/test_gen3_workflow.py 26 2 1 29
tests/test_dbgap.py 4 0 1 5
tests/test_user_data_library.py 0 0 4 4
tests/test_study_viewer.py 0 0 3 3
tests/test_data_upload.py 0 0 9 9
tests/test_discoverypage.py 0 0 1 1
tests/test_user_login_activation.py 2 0 1 3
tests/test_gen3ff_landing_page.py 0 0 2 2
tests/test_register_user.py 0 0 2 2
tests/test_aggregate_mds.py 0 0 1 1
tests/test_google_data_access.py 0 0 1 1
tests/test_env_sanity.py 0 0 1 1
tests/test_etl.py 0 0 1 1
tests/test_gen3client.py 0 0 1 1
tests/test_gen3_sdk.py 0 0 1 1
tests/test_centralized_auth.py 16 0 0 16
tests/test_presigned_url.py 7 0 0 7
tests/test_requestor.py 6 0 0 6
tests/test_drs_endpoint.py 6 0 0 6
tests/test_user_token.py 5 0 0 5
tests/test_login_page.py 2 0 0 2
tests/test_oidc_client.py 2 0 0 2
tests/test_homepage.py 1 0 0 1
tests/test_client_credentials.py 1 0 0 1
tests/test_fence_admin.py 2 0 0 2
tests/test_workspace.py 0 1 0 1
tests/test_manifest_service.py 0 1 0 1
TOTAL 92 8 73 173

Test summary after rerunning failed integration tests

filepath passed failed SUBTOTAL
tests/test_graph_submit_and_query.py 1 0 1
tests/test_gen3_workflow.py 2 0 2
tests/test_workspace.py 0 1 1
tests/test_audit_service.py 0 3 3
tests/test_manifest_service.py 0 1 1
TOTAL 3 5 8

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

@github-actions

Copy link
Copy Markdown

Integration Tests

Test summary after running integration tests

filepath failed skipped SUBTOTAL
tests/test_manifest_service.py 1 0 1
tests/test_workspace.py 1 0 1
tests/test_audit_service.py 3 3 6
TOTAL 5 3 8

Test summary after rerunning failed integration tests

filepath failed SUBTOTAL
tests/test_manifest_service.py 1 1
tests/test_workspace.py 1 1
tests/test_audit_service.py 3 3
TOTAL 5 5

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

@nss10

nss10 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Skipping Audit service test since the test is failing due to network policies and has to be dealt as a separate issue.

@github-actions

Copy link
Copy Markdown

Integration Tests

Test summary after running integration tests

filepath failed SUBTOTAL
tests/test_manifest_service.py 1 1
tests/test_workspace.py 1 1
TOTAL 2 2

Test summary after rerunning failed integration tests

filepath failed SUBTOTAL
tests/test_workspace.py 1 1
tests/test_manifest_service.py 1 1
TOTAL 2 2

Please find the detailed integration test report here

Please find the detailed integration test report after rerunning failed tests here

Please find the Github Action logs here

@nss10

nss10 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Re-running CI with just TestHomePage, since the tests currently failing are not caused by the current changes. Tested them in a different dummy PR (#1845)

@github-actions

Copy link
Copy Markdown

Integration Tests

filepath passed SUBTOTAL
tests/test_homepage.py 1 1
TOTAL 1 1

Please find the detailed integration test report here

Please find the Github Action logs here

@nss10
nss10 requested a review from Avantol13 July 15, 2026 17:24

@nrandorf nrandorf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All checks passing.

@nss10
nss10 merged commit ab2ace1 into master Jul 15, 2026
2 checks passed
@nss10
nss10 deleted the chore/update_fence_brhstaging branch July 15, 2026 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants