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

Update f35b #93

Merged
merged 4 commits into from
Nov 19, 2021
Merged

Update f35b #93

merged 4 commits into from
Nov 19, 2021

Conversation

cevich
Copy link
Member

@cevich cevich commented Sep 28, 2021

Depends on: #91 #92 #94 #98

Note: Deployment of these images need to include attention to re-introduction of auto-resize on both Fedora VMs.

@cevich cevich force-pushed the update_F35b branch 6 times, most recently from 9cb6f3c to f9f484f Compare September 29, 2021 19:11
@github-actions
Copy link

Cirrus CI build successful. Image ID c5323094450503680 ready for use.

@cevich cevich marked this pull request as ready for review September 29, 2021 20:48
@cevich cevich force-pushed the update_F35b branch 3 times, most recently from c0821ab to 1fcc0a7 Compare September 30, 2021 16:35
@github-actions
Copy link

Cirrus CI build successful. Image ID c5714136056201216 ready for use.

@cevich cevich force-pushed the update_F35b branch 3 times, most recently from 9d70f60 to 927a6a9 Compare October 1, 2021 14:01
@github-actions
Copy link

github-actions bot commented Oct 1, 2021

Cirrus CI build successful. Image ID c4556329105752064 ready for use.

2 similar comments
@github-actions
Copy link

github-actions bot commented Oct 1, 2021

Cirrus CI build successful. Image ID c4556329105752064 ready for use.

@github-actions
Copy link

github-actions bot commented Oct 1, 2021

Cirrus CI build successful. Image ID c4556329105752064 ready for use.

@cevich cevich mentioned this pull request Oct 5, 2021
@cevich cevich force-pushed the update_F35b branch 10 times, most recently from 74379bc to db3442c Compare October 7, 2021 16:17
@github-actions
Copy link

Cirrus CI build successful. Image ID c4978731958665216 ready for use.

@github-actions
Copy link

Cirrus CI build successful. Image ID c6033361849286656 ready for use.

@github-actions
Copy link

Cirrus CI build successful. Image ID c5993382683082752 ready for use.

@github-actions
Copy link

github-actions bot commented Nov 4, 2021

Cirrus CI build successful. Image ID c5920205063847936 ready for use.

@github-actions
Copy link

github-actions bot commented Nov 8, 2021

Cirrus CI build successful. Image ID c6662085032542208 ready for use.

@cevich cevich force-pushed the update_F35b branch 5 times, most recently from 41a0b9e to 4d25f8d Compare November 10, 2021 16:17
@github-actions
Copy link

Cirrus CI build successful. Image ID c4656216254185472 ready for use.

@github-actions
Copy link

Cirrus CI build successful. Image ID c6305968288956416 ready for use.

@github-actions
Copy link

Cirrus CI build successful. Image ID c5866995854868480 ready for use.

In GCP, there are google services that must start on boot but we also
rely on the more generic cloud-init services to setup the VM.
Specifically, this is necessary to attend to rootfs resizing properly
before testing.  However, there is a dependency loop created when both
are installed, that will result in messages like:

...network-online.target: Found ordering cycle on \
    cloud-init.service/start
...network-online.target: Found dependency on \
    google-network-daemon.service/start
...network-online.target: Found dependency on \
    google-instance-setup.service/start
...network-online.target: Found dependency on \
    network-online.target/start
...network-online.target: Job cloud-init.service/start \
    deleted to break ordering cycle starting with \
    network-online.target/start

Previously, this was addressed by depositing a customized
cloud-init.service file for Fedora VMs.  However, at some point in the
past this step was dropped.  Restore it.

Also, add a header-comment to this and another custom service file to
indicate where they came from.

Signed-off-by: Chris Evich <[email protected]>
@github-actions
Copy link

Cirrus CI build successful. Image ID c5709498867253248 ready for use.

The podman build process will install this if it's missing for any
reason.  That could result in inconsistent CI-vs-user experience.
Installing it explicitly (here) forces new version requirements to
come through the packaging system.

Signed-off-by: Chris Evich <[email protected]>
Older versions of podman (<3.4.1) allowed logging into a registry
using a FQIN.  Newer versions do not.  Fix this by stripping off any tag
present in the FQIN while logging in.

Signed-off-by: Chris Evich <[email protected]>
The F35 cloud image is now based on a BTRFS root filesystem.  Since
secondary storage provisioning is no longer tested by containers/storage
support for runtime storage-expansion in Fedora images is being dropped.
This significantly simplifies the build and runtime, but requires all
repos using these images to remove calls to `growpart` and other runtime
partitioning scripts.

Signed-off-by: Chris Evich <[email protected]>
@github-actions
Copy link

Cirrus CI build successful. Image ID c6226133906620416 ready for use.

@cevich cevich merged commit 438ba2f into containers:main Nov 19, 2021
@cevich cevich deleted the update_F35b branch July 19, 2022 14:45
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.

1 participant