Skip to content
This repository has been archived by the owner on Dec 19, 2024. It is now read-only.

deprecate repository #2256

Merged
merged 1 commit into from
Dec 12, 2024
Merged

deprecate repository #2256

merged 1 commit into from
Dec 12, 2024

Conversation

ktdreyer
Copy link
Contributor

@ktdreyer ktdreyer commented Dec 3, 2024

Remove all the files in the repository and point users at the new Containerfile location.

@guits
Copy link
Collaborator

guits commented Dec 4, 2024

@ktdreyer I wonder if removing everything might be a bit too "aggressive" for now. Shouldn't we just update the README.md to inform people that the repository is deprecated and redirect them to the Containerfile location?

@ktdreyer
Copy link
Contributor Author

ktdreyer commented Dec 4, 2024

The files will still be in the Git history. Removing the files makes it clear to users and contributors that developers are focused on the new area.

@travisn
Copy link
Member

travisn commented Dec 4, 2024

Why not just archive the repo so it's readonly? And how will they know where to find new development if the readme is even empty?

@guits
Copy link
Collaborator

guits commented Dec 4, 2024

And how will they know where to find new development if the readme is even empty?

Correct. Although I still think that removing everything is a bit aggressive, I would at least leave a comment in README.md to let users know what's happening.

@ktdreyer
Copy link
Contributor Author

ktdreyer commented Dec 4, 2024

Whoops, sorry! I wrote a new README but I forgot to push it in this PR. I've amended and re-pushed. It can be a little difficult to read in this large GitHub diff, so I'll paste the README contents here:


Containerfile has moved

This ceph-container GitHub repository is deprecated and read-only. The Ceph developers have moved all development to the Containerfile in the primary ceph GitHub repository:

Older releases:

Please make future contributions to these locations.

History

This repository originally held the first experiments of putting Ceph into container images. Later on, this repository held a variety of operating systems and versions in file snippets with a templating system. We have simplified this to a single flat Containerfile, one per Ceph release.

README.md Outdated Show resolved Hide resolved
Remove all the files in the repository and point users at the new
Containerfile location.

Signed-off-by: Ken Dreyer <[email protected]>
@ktdreyer
Copy link
Contributor Author

ktdreyer commented Dec 9, 2024

@dmick , is this ready to merge now?

guits added a commit to ceph/ceph-build that referenced this pull request Dec 9, 2024
These jobs are no longer needed, see ceph/ceph-container#2256

Signed-off-by: Guillaume Abrioux <[email protected]>
@dmick
Copy link
Member

dmick commented Dec 9, 2024 via email

@ktdreyer
Copy link
Contributor Author

Now that ceph/ceph-build#2305 is done, @dmick is this read to merge?

@dmick
Copy link
Member

dmick commented Dec 12, 2024

I think so. I think ceph/ceph#60936 is too if I could get a review 😄

@dmick dmick merged commit 139b31d into main Dec 12, 2024
4 checks passed
@dmick dmick deleted the remove-all branch December 12, 2024 18:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants