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

Running single container for nodes containers #204

Merged
merged 2 commits into from
Sep 11, 2023

Conversation

Yarboa
Copy link
Collaborator

@Yarboa Yarboa commented Sep 11, 2023

Resolve images issue in #164

This PR fix the creating multi images for many nodes,
Now all node container should use node image.

@Yarboa Yarboa self-assigned this Sep 11, 2023
@Yarboa Yarboa marked this pull request as draft September 11, 2023 07:17
tests/e2e/lib/container Outdated Show resolved Hide resolved
@Yarboa Yarboa force-pushed the use-single-image branch 3 times, most recently from 831cfda to b64e730 Compare September 11, 2023 08:24
Copy link
Collaborator Author

@Yarboa Yarboa left a comment

Choose a reason for hiding this comment

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

Please add check if images alredy exist, to avoid trying to recreate images

Resolve containers#164
nodeX images

Signed-off-by: Yariv Rachmani <yrachman@redhat.com>
Signed-off-by: Yariv Rachmani <yrachman@redhat.com>
@Yarboa Yarboa marked this pull request as ready for review September 11, 2023 10:21
@lsm5
Copy link
Member

lsm5 commented Sep 11, 2023

packit blocked on fedora-copr/copr#2906 . Might wanna ignore packit builds until then if the PR is known to not interfere with rpm builds.

@rhatdan
Copy link
Member

rhatdan commented Sep 11, 2023

LGTM

@Yarboa
Copy link
Collaborator Author

Yarboa commented Sep 11, 2023

packit blocked on fedora-copr/copr#2906 . Might wanna ignore packit builds until then if the PR is known to not interfere with rpm builds.

Seems it is fixed :)

@dougsland dougsland merged commit 1e7bf2a into containers:main Sep 11, 2023
6 checks passed
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.

4 participants