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

[ZKUI-349] ✨ Implement GHAS #586

Merged

Conversation

gaspardmoindrot
Copy link
Contributor

No description provided.

@gaspardmoindrot gaspardmoindrot self-assigned this May 15, 2023
@bert-e
Copy link
Contributor

bert-e commented May 15, 2023

Hello gaspardmoindrot,

My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.

Status report is not available.

@bert-e
Copy link
Contributor

bert-e commented May 15, 2023

Incorrect fix version

The Fix Version/s in issue ZKUI-349 contains:

  • None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

  • 1.6.14

  • 2.0.4

Please check the Fix Version/s of ZKUI-349, or the target
branch of this pull request.

@codecov-commenter
Copy link

codecov-commenter commented May 16, 2023

Codecov Report

Merging #586 (5324cfe) into development/1.6 (97277c3) will decrease coverage by 0.24%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##           development/1.6     #586      +/-   ##
===================================================
- Coverage            55.39%   55.16%   -0.24%     
===================================================
  Files                  218      218              
  Lines                10352    10352              
  Branches              2907     2907              
===================================================
- Hits                  5735     5711      -24     
- Misses                4599     4623      +24     
  Partials                18       18              

see 6 files with indirect coverage changes

permissions:
contents: read
packages: write
steps:
Copy link
Contributor

Choose a reason for hiding this comment

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

as this repo is not using any kind of internally made reuasable workflow, feel free to use the one on scality/workflows.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is it better now ?

.github/workflows/security.yaml Outdated Show resolved Hide resolved
@gaspardmoindrot gaspardmoindrot force-pushed the improvement/ZKUI-349-implement-ghas branch from 641afed to 04bbe25 Compare May 16, 2023 01:21
Copy link
Contributor

@JBWatenbergScality JBWatenbergScality left a comment

Choose a reason for hiding this comment

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

Hello, Thanks for your contribution, few comments on my side.

.github/workflows/security.yaml Outdated Show resolved Hide resolved
Comment on lines +17 to +23
build-zenko-ui:
permissions:
contents: read
packages: write
uses: scality/workflows/.github/workflows/docker-build.yaml@v1
with:
name: zenko-ui
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of rebuilding the image, we can run the scan on the image matching the head of the branch.

Copy link
Contributor

Choose a reason for hiding this comment

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

We might be facing a race condition in this case. If we want this workflow to be independent we should be building this image.

Copy link
Contributor Author

@gaspardmoindrot gaspardmoindrot May 16, 2023

Choose a reason for hiding this comment

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

You mean using the build that is created in build-push-tests.yml, right? In this case, indeed, we may have a race condition issue since we cannot use cross-file needs.

Copy link
Contributor

@JBWatenbergScality JBWatenbergScality left a comment

Choose a reason for hiding this comment

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

Hello, Thanks for your contribution, few comments on my side.

@bert-e
Copy link
Contributor

bert-e commented May 16, 2023

Incorrect fix version

The Fix Version/s in issue ZKUI-349 contains:

  • None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

  • 1.6.14

  • 2.0.5

Please check the Fix Version/s of ZKUI-349, or the target
branch of this pull request.

@gaspardmoindrot gaspardmoindrot force-pushed the improvement/ZKUI-349-implement-ghas branch from 6c1c68d to cd3b0f8 Compare May 16, 2023 16:59
@bert-e
Copy link
Contributor

bert-e commented May 17, 2023

Incorrect fix version

The Fix Version/s in issue ZKUI-349 contains:

  • None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

  • 1.6.15

  • 2.0.5

Please check the Fix Version/s of ZKUI-349, or the target
branch of this pull request.

@gaspardmoindrot
Copy link
Contributor Author

ping

@bert-e
Copy link
Contributor

bert-e commented May 18, 2023

Incorrect fix version

The Fix Version/s in issue ZKUI-349 contains:

  • 1.6.14

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

  • 1.6.15

  • 2.0.5

Please check the Fix Version/s of ZKUI-349, or the target
branch of this pull request.

@gaspardmoindrot
Copy link
Contributor Author

ping

@bert-e
Copy link
Contributor

bert-e commented May 18, 2023

Integration data created

I have created the integration data for the additional destination branches.

The following branches will NOT be impacted:

  • development/1.4
  • development/1.5

You can set option create_pull_requests if you need me to create
integration pull requests in addition to integration branches, with:

@bert-e create_pull_requests

@bert-e
Copy link
Contributor

bert-e commented May 18, 2023

Waiting for approval

The following approvals are needed before I can proceed with the merge:

  • the author

  • one peer

Peer approvals must include at least 1 approval from the following list:

@gaspardmoindrot gaspardmoindrot merged commit a9dc167 into development/1.6 May 18, 2023
@gaspardmoindrot gaspardmoindrot deleted the improvement/ZKUI-349-implement-ghas branch May 18, 2023 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants