-
Notifications
You must be signed in to change notification settings - Fork 77
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
ci: Fix issue with github_env echo statements #3012
ci: Fix issue with github_env echo statements #3012
Conversation
**Details**: Updates the OPERATOR_KEY_MAIN environment variable storage to use a masked value. **Related Issues**: Fixes: #3011 Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
level env section. Removing the need for the manual assign step in acceptance-workflow Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was able to test this with the on_demand workflow dispatch and it looks good!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3012 +/- ##
==========================================
+ Coverage 88.33% 89.92% +1.59%
==========================================
Files 61 58 -3
Lines 4054 3900 -154
Branches 815 780 -35
==========================================
- Hits 3581 3507 -74
+ Misses 430 346 -84
- Partials 43 47 +4
Flags with carried forward coverage won't be shown. Click here to find out more. |
Details:
Updates the OPERATOR_KEY_MAIN environment variable storage to use a masked value.
Related Issues:
Fixes: #3011