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

SIMSBIOHUB-540/528: Pipeline Enhancements #1204

Draft
wants to merge 70 commits into
base: dev
Choose a base branch
from

Conversation

NickPhura
Copy link
Collaborator

@NickPhura NickPhura commented Jan 25, 2024

May hold off on merging this until after we go to Prod, just to ensure the initial prod release works without issue under the old pipeline, which we know works.

Links to Jira Tickets

https://apps.nrs.gov.bc.ca/int/jira/browse/SIMSBIOHUB-540
https://apps.nrs.gov.bc.ca/int/jira/browse/SIMSBIOHUB-528

Description of Changes

Moving pipeline config settings into OpenShift config map, so they aren't hard-coded in the repo. Will allow env var changes to be made without needing to commit anything to the repo.

Testing Notes

The pipeline deploys, and all previous functionality works as expected.

@NickPhura NickPhura added the Do Not Merge PR should not be merged label Jan 25, 2024
Copy link

codecov bot commented Jan 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.43%. Comparing base (ce90f56) to head (855edaf).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #1204   +/-   ##
=======================================
  Coverage   56.43%   56.43%           
=======================================
  Files         576      576           
  Lines       17626    17626           
  Branches     2745     2745           
=======================================
  Hits         9948     9948           
  Misses       7014     7014           
  Partials      664      664           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@NickPhura NickPhura added the Not Ready For Review Addressing feedback and/or refactoring label Jan 25, 2024
curtisupshall
curtisupshall previously approved these changes Mar 19, 2024
Copy link
Contributor

@curtisupshall curtisupshall left a comment

Choose a reason for hiding this comment

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

Codes looks good to me 👍

curtisupshall
curtisupshall previously approved these changes Mar 19, 2024
@NickPhura NickPhura added the Do Not Merge PR should not be merged label Mar 19, 2024
Copy link

sonarcloud bot commented Mar 22, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@NickPhura NickPhura added Early Feedback Welcome PR is not finished, but early review feedback is welcomed and removed Ready For Review PR is ready for review labels Mar 27, 2024
@NickPhura NickPhura marked this pull request as draft May 30, 2024 23:43
@NickPhura NickPhura removed the Early Feedback Welcome PR is not finished, but early review feedback is welcomed label May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge PR should not be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants