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

Cirrus: use image with fewer downloaded dependencies #1739

Closed
wants to merge 1 commit into from
Closed

Cirrus: use image with fewer downloaded dependencies #1739

wants to merge 1 commit into from

Conversation

lsm5
Copy link
Member

@lsm5 lsm5 commented Aug 24, 2022 •
edited
Loading

The latest cirrus image bump will use fewer downloaded dependencies in
favor of installation from copr / distro repos.

Signed-off-by: Lokesh Mandvekar [email protected]

@cevich PTAL

@lsm5
Copy link
Member Author

lsm5 commented Aug 24, 2022

ugh .. my bad RE: fedora 37

@lsm5 lsm5 force-pushed the bump-cirrus-image branch from 15f5b2b to c89f87f Compare August 24, 2022 15:51
@lsm5 lsm5 changed the title Cirrus: enable Fedora 37 Cirrus: use image with fewer downloaded dependencies Aug 24, 2022
@lsm5 lsm5 force-pushed the bump-cirrus-image branch from c89f87f to 599ff9e Compare August 24, 2022 15:53
@lsm5
Copy link
Member Author

lsm5 commented Aug 24, 2022

/hold

@lsm5 lsm5 force-pushed the bump-cirrus-image branch from 599ff9e to 4d0faf6 Compare August 24, 2022 15:58
@lsm5 lsm5 mentioned this pull request Aug 24, 2022
Merged
@lsm5
Copy link
Member Author

lsm5 commented Aug 24, 2022

@cevich hmm, make test-system-local passes in rerun with terminal. But can't see why it's failing in CI run

@lsm5 lsm5 mentioned this pull request Aug 25, 2022
Closed
@lsm5 lsm5 force-pushed the bump-cirrus-image branch 2 times, most recently from 05f8830 to ec1cbb3 Compare August 29, 2022 14:29
@lsm5
Copy link
Member Author

lsm5 commented Aug 31, 2022

@vrothberg @mtrmac could you PTAL at the skopeo test failure? Don't think it's related to cirrus image bump.

@mtrmac
Copy link
Contributor

mtrmac commented Aug 31, 2022

@lsm5 At a very quick glance this looks as if some config file was missing (or inaccessible?).

At least #1738 has passed tests during the lifetime of this PR, and containers/image#1641 is similarly suggestive that this is something to do with the image change.

@edsantiago
Copy link
Member

What version of podman is on this new system? The "missing necessary StoreOptions" message disappeared from podman in containers/podman#9734 (March 2021), and the "Storage configuration is unset" message was killed in containers/podman#4698 (December 2019) (!)

@lsm5 lsm5 force-pushed the bump-cirrus-image branch from ec1cbb3 to 1b2d5e9 Compare September 9, 2022 15:01
@lsm5 lsm5 force-pushed the bump-cirrus-image branch from 1b2d5e9 to 4554ba5 Compare September 16, 2022 19:27
@lsm5
Cirrus: use image with fewer downloaded dependencies
1361f56 
The latest cirrus image bump will use fewer downloaded dependencies in
favor of installation from copr / distro repos.

Signed-off-by: Lokesh Mandvekar <[email protected]>
@lsm5 lsm5 force-pushed the bump-cirrus-image branch from 4554ba5 to 1361f56 Compare September 29, 2022 14:17
@github-actions
Copy link

A friendly reminder that this PR had no activity for 30 days.

@mtrmac
Copy link
Contributor

mtrmac commented Dec 5, 2022

What version of podman is on this new system? The "missing necessary StoreOptions" message disappeared from podman in containers/podman#9734 (March 2021), and the "Storage configuration is unset" message was killed in containers/podman#4698 (December 2019) (!)

These messages AFAICS still exist in c/storage and c/common (not directly in Podman), as of today’s main.

@lsm5 a few months later, those failures still seem to be related to this PR. Does comparing contents of the two images show anything relevant?

@github-actions github-actions bot removed the stale-pr label Dec 6, 2022
@cevich
Copy link
Member

cevich commented Dec 6, 2022

IIRC, the main difference is just some missing packages and "newer" images. So I fear we'll see same/similar problems pop up when I get around to bringing F37 here.

@mtrmac mtrmac mentioned this pull request Dec 7, 2022
Merged
@github-actions
Copy link

github-actions bot commented Jan 6, 2023

A friendly reminder that this PR had no activity for 30 days.

@cevich
Copy link
Member

cevich commented Jan 10, 2023

I think we can close this in favor of #1821 but need @lsm5 to confirm (and merge his automation_images PR).

@github-actions github-actions bot removed the stale-pr label Jan 11, 2023
@lsm5
Copy link
Member Author

lsm5 commented Jan 27, 2023

I think we can close this in favor of #1821 but need @lsm5 to confirm (and merge his automation_images PR).

closing ..

@lsm5 lsm5 closed this Jan 27, 2023
@lsm5 lsm5 deleted the bump-cirrus-image branch June 21, 2023 14:14
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 20, 2023
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
locked - please file new issue/PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lsm5 @mtrmac @edsantiago @cevich