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

Modify scripts/{sync-pull,sign-rpms,sync-push} to handle ceph-iscsi #2243

Merged
merged 4 commits into from
May 30, 2024

Conversation

dmick
Copy link
Member

@dmick dmick commented May 22, 2024

All the tools and paths basically grow a "project" argument to distinguish ceph from ceph-iscsi.

In addition, drop el7, add vim modelines, and other minor cleanup.

dmick added 4 commits May 22, 2024 13:30
Signed-off-by: Dan Mick <dan.mick@redhat.com>
This allows signing ceph-iscsi packages

Signed-off-by: Dan Mick <dan.mick@redhat.com>
Several sections are done only for the 'ceph' project (to
accommodate ceph-iscsi, which has no debs, so doesn't need
to find the most-populated distroversion or create debs,
pull a tarball, etc.).

Also drop el7.

Signed-off-by: Dan Mick <dan.mick@redhat.com>
ceph-iscsi also needs a repo file created
Drop el7
Add usage

Signed-off-by: Dan Mick <dan.mick@redhat.com>
@dmick dmick requested review from jdurgin and ktdreyer May 22, 2024 22:16
@dmick
Copy link
Member Author

dmick commented May 22, 2024

This requires ceph/ceph-container#2210 to use the new paths

@dmick
Copy link
Member Author

dmick commented May 22, 2024

This was tested for constructing the release of ceph-iscsi 3.8 and ceph 18.2.3

dmick added a commit to dmick/ceph that referenced this pull request May 22, 2024
Support added to the release scripts (from ceph-build.git) to
work for ceph-iscsi, so 'project' must be passed to these scripts,
and will appear in the prerelease pathnames.  See also
ceph/ceph-build#2243 and
ceph/ceph-container#2210

Signed-off-by: Dan Mick <dan.mick@redhat.com>
dmick added a commit to dmick/ceph that referenced this pull request May 22, 2024
Support added to the release scripts (from ceph-build.git) to
work for ceph-iscsi, so 'project' must be passed to these scripts,
and will appear in the prerelease pathnames.  See also
ceph/ceph-build#2243 and
ceph/ceph-container#2210

Signed-off-by: Dan Mick <dan.mick@redhat.com>
zdover23 pushed a commit to zdover23/ceph that referenced this pull request May 23, 2024
Support added to the release scripts (from ceph-build.git) to
work for ceph-iscsi, so 'project' must be passed to these scripts,
and will appear in the prerelease pathnames.  See also
ceph/ceph-build#2243 and
ceph/ceph-container#2210

Signed-off-by: Dan Mick <dan.mick@redhat.com>
(cherry picked from commit 7e64c63)
zdover23 pushed a commit to zdover23/ceph that referenced this pull request May 23, 2024
Support added to the release scripts (from ceph-build.git) to
work for ceph-iscsi, so 'project' must be passed to these scripts,
and will appear in the prerelease pathnames.  See also
ceph/ceph-build#2243 and
ceph/ceph-container#2210

Signed-off-by: Dan Mick <dan.mick@redhat.com>
(cherry picked from commit 7e64c63)
zdover23 pushed a commit to zdover23/ceph that referenced this pull request May 23, 2024
Support added to the release scripts (from ceph-build.git) to
work for ceph-iscsi, so 'project' must be passed to these scripts,
and will appear in the prerelease pathnames.  See also
ceph/ceph-build#2243 and
ceph/ceph-container#2210

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

dmick commented May 29, 2024

retest

@dmick
Copy link
Member Author

dmick commented May 30, 2024

The test is failing, but there are no tests for these scripts (the test failure is unrelated). Merging.

@dmick dmick merged commit ceb47c8 into ceph:main May 30, 2024
1 check failed
@dmick dmick deleted the wip-scripts-ceph-iscsi branch May 30, 2024 09:07
mkogan1 pushed a commit to mkogan1/ceph that referenced this pull request Aug 7, 2024
Support added to the release scripts (from ceph-build.git) to
work for ceph-iscsi, so 'project' must be passed to these scripts,
and will appear in the prerelease pathnames.  See also
ceph/ceph-build#2243 and
ceph/ceph-container#2210

Signed-off-by: Dan Mick <dan.mick@redhat.com>
(cherry picked from commit 7e64c63)
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.

2 participants