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.

Sign up for GitHub

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

Jump to bottom

[DO NOT MERGE] Test vendor c/common CNI build tag #21179

Closed
wants to merge 2 commits into from
Closed

[DO NOT MERGE] Test vendor c/common CNI build tag #21179

wants to merge 2 commits into from

Conversation

ashley-cui
Copy link
Member

@ashley-cui ashley-cui commented Jan 5, 2024 •
edited
Loading

Podman test PR for build-tagging out CNI.

Relevant c/common PR: containers/common#1767

@ashley-cui ashley-cui added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 5, 2024
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 5, 2024
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 5, 2024

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci openshift-ci bot added the do-not-merge/release-note-label-needed Enforce release-note requirement, even if just None label Jan 5, 2024
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 5, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ashley-cui

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 5, 2024
@ashley-cui ashley-cui marked this pull request as draft January 5, 2024 16:41
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 5, 2024
@packit-as-a-service Packit-as-a-Service
Copy link

Cockpit tests failed for commit 7153da1fc1368fbf96e5d8bfc4805dece6a7784a. @martinpitt, @jelly, @mvollmer please check.

@ashley-cui
Copy link
Member Author

Bunch of CI stuff needs to be edited - hang tight

@martinpitt
Copy link
Contributor

@ashley-cui : Do you mean this error?

+ podman pull quay.io/prometheus/busybox
Error: failed to get default network backend: unknown network backend value "" in "/var/lib/containers/storage/defaultNetworkBackend"

@ashley-cui ashley-cui force-pushed the cni branch 4 times, most recently from 5bba41c to 2d4cc4c Compare January 22, 2024 01:50
@packit-as-a-service Packit-as-a-Service
Copy link

Cockpit tests failed for commit 54bdece6f7105ada9101787cde0851937712b02f. @martinpitt, @jelly, @mvollmer please check.

@packit-as-a-service Packit-as-a-Service
Copy link

Cockpit tests failed for commit d19f1c0b46df6a0e5fb80e3976ddfc5a0e34099c. @martinpitt, @jelly, @mvollmer please check.

@packit-as-a-service Packit-as-a-Service
Copy link

Cockpit tests failed for commit bcd9db93e43500b444007e950b81370f03000597. @martinpitt, @jelly, @mvollmer please check.

@packit-as-a-service Packit-as-a-Service
Copy link

Cockpit tests failed for commit d19f1c0b46df6a0e5fb80e3976ddfc5a0e34099c. @martinpitt, @jelly, @mvollmer please check.

@packit-as-a-service Packit-as-a-Service
Copy link

Cockpit tests failed for commit 5bba41ca86ff3fa65f1e036dfcbaffaa7f05a743. @martinpitt, @jelly, @mvollmer please check.

@packit-as-a-service Packit-as-a-Service
Copy link

Cockpit tests failed for commit 2d4cc4c9eb1fc60b6622a201229f98792bd9f2de. @martinpitt, @jelly, @mvollmer please check.

@ashley-cui ashley-cui force-pushed the cni branch 2 times, most recently from 63426a7 to 2914da7 Compare January 22, 2024 18:36
@packit-as-a-service Packit-as-a-Service
Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

@packit-as-a-service Packit-as-a-Service
Copy link

Cockpit tests failed for commit 2914da77fb1302dd970a09e9c39a73736bf20fde. @martinpitt, @jelly, @mvollmer please check.

@ashley-cui ashley-cui mentioned this pull request Jan 31, 2024
Merged
@ashley-cui ashley-cui force-pushed the cni branch 5 times, most recently from de0177f to bb43b7f Compare January 31, 2024 16:44
@ashley-cui
Temporarily build with CNI on F38
1d5f2a2 
Fixes c/common vendoring on main until VM's are updated

Signed-off-by: Ashley Cui <[email protected]>
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 31, 2024
@openshift-merge-robot
Copy link
Collaborator

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ashley-cui ashley-cui closed this Feb 1, 2024
@stale-locking-app stale-locking-app bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label May 2, 2024
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators May 2, 2024
@ashley-cui ashley-cui deleted the cni branch June 17, 2024 14:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/release-note-label-needed Enforce release-note requirement, even if just None do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ashley-cui @martinpitt @openshift-merge-robot