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

drop debugs #2092

Merged
merged 3 commits into from
Nov 15, 2024
Merged

drop debugs #2092

merged 3 commits into from
Nov 15, 2024

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented Jun 2, 2024

No description provided.

@@ -36,7 +35,6 @@
'/var/www/vhosts/web/htdocs',
'/var/www/vhosts/archivedeb/htdocs',
'/var/www/vhosts/deb/htdocs',
'/var/www/vhosts/debugs/htdocs',
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would manually clean up this folder so it doesn't accidentally leak data that is currently there and behind password.

@evgeni evgeni marked this pull request as ready for review November 13, 2024 16:12
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We had these RFCs in the past:
https://community.theforeman.org/t/rfc-deprecate-foreman-debug/13361
https://community.theforeman.org/t/rfc-deprecate-foreman-debug-in-favor-of-sosreport/12465

I can't find a concrete deprecation planning and would like to see such post, tying this all back. In addition to that, we can drop the debug subpackage in foreman.spec and probably the Debian equivalent.

@evgeni
Copy link
Member Author

evgeni commented Nov 13, 2024

The reality is, no new content could be uploaded to debugs.tfm.o since https://community.theforeman.org/t/sunsetting-rsync-theforeman-org/37884

So removing the web-part of the setup doesn't require a post as such, as it was only ever used by us to obtain debug tarballs.

(It's a total coincidence that the original submission of this PR was almost the exact day we turned off rsync.tfm.o ;) )

@evgeni evgeni merged commit df28da9 into master Nov 15, 2024
2 checks passed
@evgeni evgeni deleted the drop-debugs branch November 15, 2024 06:59
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.

3 participants