#1280 Have container-nightly.sh publish all container images
Merged 17 days ago by kevin. Opened 17 days ago by adamwill.
adamwill/pungi-fedora container-nightly-sync-all-containers  into  main

file modified
+4 -1
@@ -77,9 +77,12 @@ 

  #         mutt -e "set from=\"$FROM\"" -e 'set envelope_from=yes' -s "$SUBJECT" $tomail

  #done

  

- # Push rawhide ostree base container image to fedora registry

  pushd ./releng

  pushd ./scripts

+ # Publish OCI container images we just built to Fedora and Quay registries

+ ./sync-latest-container-base-image.sh $RELEASE

+ # Also convert latest Atomic Desktop ostrees to OCIs and publish those, even

+ # though this script does not build them (because Bodhi cannot do this itself)

  ./sync-ostree-base-containers.sh $RELEASE

  popd

  popd

What container-nightly.sh does to publish container images is
currently inconsistent across branches. #1267 made it publish
only the regular container images (the ones it actually builds)
on the f39 branch. On f40 and main branches, it currently would
not publish the images it builds at all, it instead only
runs sync-ostree-base-containers.sh , which converts the ostrees
for some of the atomic desktops to OCI images and publishes
those (which is also what it used to do on the f39 branch).

We should make the behavior consistent across all branches. The
script is never actually run on the 'main' branch, but new
releases branch off 'main' so we should keep it in line with
our intentions.

Overall at present, I think the most sensible thing to do is
have this script run both sync scripts.

The Fedora-Container compose does not build atomic desktop stuff
at all; Bodhi does that. But Bodhi does not have any ability to
convert the ostrees it builds to OCI images, natively produce
OCI images, or publish OCI images to registries. So if we don't
have something else do that, we will never publish updated OCI
images for atomic desktops for stable releases. Since we ought
to run the script somewhere, doing it here seems as good as
anywhere.

Longer term we need to make various improvements to this whole
process, but for now this seems the pragmatic thing to do.

Signed-off-by: Adam Williamson awilliam@redhat.com

ok. Yes, I am fine with doing this as a short term thing.

Pull-Request has been merged by kevin

17 days ago
Metadata