Skip to content

reef: Add Containerfile and build.sh to build it.#60228

Merged
idryomov merged 2 commits intoceph:reeffrom
dmick:wip-68468-reef
Oct 15, 2024
Merged

reef: Add Containerfile and build.sh to build it.#60228
idryomov merged 2 commits intoceph:reeffrom
dmick:wip-68468-reef

Conversation

@dmick
Copy link
Member

@dmick dmick commented Oct 9, 2024

backport tracker: https://tracker.ceph.com/issues/68468


backport of #59868 and #60255
parent tracker: https://tracker.ceph.com/issues/68467

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

The intent is to replace ceph-container.git, at first for ci containers
only, and eventually production containers as well.

There is code present for production containers, including
a separate "make-manifest-list.py" to scan for and glue the two
arch-specific containers into a 'manifest-list' 'fat' container,
but that code is not yet fully tested.

This code will not be used until a corresponding change to the
Jenkins jobs in ceph-build.git is pushed.

Note that this tooling does not authenticate to the container repo;
it is assumed that will be done elsewhere.  Authentication is
verified by pushing a minimal image to the requested repo.

Signed-off-by: Dan Mick <dmick@redhat.com>
(cherry picked from commit 5c40a5c)
@dmick dmick added this to the reef milestone Oct 9, 2024
@dmick dmick requested a review from ljflores October 9, 2024 22:15
@dmick
Copy link
Member Author

dmick commented Oct 10, 2024

jenkins test make check

@idryomov
Copy link
Contributor

Noting that the lack of this backport is causing CentOS Stream builds to fail as follows:

/tmp/jenkins13082275400505746526.sh: line 2024: cd: /home/jenkins-build/build/workspace/ceph-dev-new-build/ARCH/x86_64/AVAILABLE_ARCH/x86_64/AVAILABLE_DIST/centos9/DIST/centos9/MACHINE_SIZE/gigantic/dist/ceph-18.2.4-942-gfd36bce9/container: No such file or directory

@idryomov
Copy link
Contributor

jenkins test make check

@ljflores
Copy link
Member

Need to also add #60255

The wrong string was used for comparison, and for tagging,
so the arm64 branch and sha1 images overwrote and destroyed
the amd64 images.

Signed-off-by: Dan Mick <dmick@redhat.com>
(cherry picked from commit 7e03ee7)
@ljflores
Copy link
Member

jenkins test api

@ljflores
Copy link
Member

jenkins test make check

@idryomov
Copy link
Contributor

Agent went offline during the build

@idryomov
Copy link
Contributor

jenkins test make check

@idryomov idryomov merged commit 7d07a97 into ceph:reef Oct 15, 2024
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