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

OCP4: Fix pr image workflow #11533

Merged
merged 1 commit into from
Feb 6, 2024
Merged

OCP4: Fix pr image workflow #11533

merged 1 commit into from
Feb 6, 2024

Conversation

Vincent056
Copy link
Contributor

@Vincent056 Vincent056 commented Feb 2, 2024

There was an issue with the existing pr image build workflow, GitHub does not let workflow use metal-toolbox/container-push/.github/workflows/container-push.yml@main in the steps, this PR replaced metal-toolbox to build and push images directly.

Copy link

github-actions bot commented Feb 2, 2024

Start a new ephemeral environment with changes proposed in this pull request:

Fedora Environment
Open in Gitpod

Oracle Linux 8 Environment
Open in Gitpod

@Vincent056
Copy link
Contributor Author

We don't have a way to test this workflow until we have this merged

@yuumasato yuumasato self-assigned this Feb 2, 2024
Copy link
Member

@yuumasato yuumasato left a comment

Choose a reason for hiding this comment

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

.github/workflows/k8s-content-pr.yaml Outdated Show resolved Hide resolved
There was a issue with existing pr image build workflow, github does not let workflow to use metal-toolbox/container-push/.github/workflows/container-push.yml@main in the steps, this PR replace metal-toolbox to build and push image directly
Copy link

codeclimate bot commented Feb 5, 2024

Code Climate has analyzed commit 7d9ba83 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 58.3% (0.0% change).

View more on Code Climate.

Copy link
Member

@yuumasato yuumasato left a comment

Choose a reason for hiding this comment

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

LGTM

@yuumasato yuumasato added this to the 0.1.73 milestone Feb 6, 2024
@yuumasato yuumasato added the OpenShift OpenShift product related. label Feb 6, 2024
@yuumasato
Copy link
Member

This PR should not be affecting any build or test in centos-stream-9

@yuumasato yuumasato merged commit 979998a into master Feb 6, 2024
47 of 52 checks passed
@yuumasato yuumasato deleted the fix_pr_image_workflow branch February 6, 2024 15:22
@Mab879 Mab879 added the Infrastructure Our content build system label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infrastructure Our content build system OpenShift OpenShift product related.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants