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

ci: Fix issue with github_env echo statements #3012

Merged

Conversation

rbarker-dev
Copy link
Contributor

Details:

Updates the OPERATOR_KEY_MAIN environment variable storage to use a masked value.

Related Issues:

Fixes: #3011

**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>
@rbarker-dev rbarker-dev added this to the 0.57.0 milestone Sep 18, 2024
@rbarker-dev rbarker-dev requested a review from beeradb September 18, 2024 15:31
@rbarker-dev rbarker-dev self-assigned this Sep 18, 2024
@rbarker-dev rbarker-dev linked an issue Sep 18, 2024 that may be closed by this pull request
Copy link

github-actions bot commented Sep 18, 2024

Acceptance Tests

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0

Results for commit d80c7d5.

♻️ This comment has been updated with latest results.

Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Copy link

github-actions bot commented Sep 18, 2024

Tests

       3 files     297 suites   19s ⏱️
1 353 tests 1 352 ✔️ 1 💤 0
1 362 runs  1 361 ✔️ 1 💤 0

Results for commit d80c7d5.

♻️ This comment has been updated with latest results.

Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
@rbarker-dev rbarker-dev marked this pull request as ready for review September 18, 2024 16:25
@rbarker-dev rbarker-dev requested review from JeffreyDallas, jeromy-cannon and nathanklick and removed request for JeffreyDallas September 18, 2024 16:25
.github/workflows/conformity-workflow.yml Outdated Show resolved Hide resolved
.github/workflows/conformity-workflow.yml Outdated Show resolved Hide resolved
.github/workflows/conformity-workflow.yml Outdated Show resolved Hide resolved
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>
Copy link

Copy link

Test Results

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0

Results for commit d80c7d5.

Copy link
Collaborator

@beeradb beeradb left a 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!

@rbarker-dev rbarker-dev merged commit acd8f4a into main Sep 19, 2024
25 of 53 checks passed
@rbarker-dev rbarker-dev deleted the 3011-update-workflows-to-mask-operator-key-when-manually-used branch September 19, 2024 20:19
Copy link

codecov bot commented Sep 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.92%. Comparing base (b160863) to head (d80c7d5).
Report is 4 commits behind head on main.

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     
Flag Coverage Δ
relay 90.14% <ø> (ø)
server 88.15% <ø> (ø)
ws-server 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 6 files with indirect coverage changes

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.

Update workflows to mask operator key when manually used
6 participants