Skip to content

Remove an obsolete foreman-rake troubleshooting advice#5082

Merged
aneta-petrova merged 1 commit into
theforeman:masterfrom
aneta-petrova:rake-migrate-deb
Jul 21, 2026
Merged

Remove an obsolete foreman-rake troubleshooting advice#5082
aneta-petrova merged 1 commit into
theforeman:masterfrom
aneta-petrova:rake-migrate-deb

Conversation

@aneta-petrova

@aneta-petrova aneta-petrova commented Jul 20, 2026

Copy link
Copy Markdown
Member

What changes are you introducing?

Removing a foreman-rake command that is obsolete per the conversation in #5075

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

Originally, the foreman-rake command was a one-time migration tool used to fix database entries when transitioning from Pulp 2 to Pulp 3. Since the Pulp 2 migration phase is over, users are unlikely to need it, and thus we can remove it from documentation.

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

N/A

Contributor checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.19/Katello 4.21
  • Foreman 3.18/Katello 4.20 (Satellite 6.19; orcharhino 7.9)
  • Foreman 3.17/Katello 4.19
  • Foreman 3.16/Katello 4.18 (Satellite 6.18; orcharhino 7.6, 7.7, and 7.8)
  • Foreman 3.15/Katello 4.17
  • Foreman 3.14/Katello 4.16 (Satellite 6.17; orcharhino 7.4; orcharhino 7.5)
  • We do not accept PRs for Foreman older than 3.14.

@github-actions github-actions Bot added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective labels Jul 20, 2026
@aneta-petrova

Copy link
Copy Markdown
Member Author

Hi @ianballou @quba42, this PR removes the foreman-rake command discussed in #5075. Can you please clarify to which versions this update should be cherry-picked? In what version did the command stop being relevant?

@github-actions

Copy link
Copy Markdown

The PR preview for f2cbe1a is available at theforeman-foreman-documentation-preview-pr-5082.surge.sh

The following output files are affected by this PR:

show diff

show diff as HTML

@quba42

quba42 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

@aneta-petrova How far do you backport? In my mind the 2to3 migration was 100 years ago, so you can probably backport to all currently supported versions. 😄

@aneta-petrova aneta-petrova added tech review done No issues from the technical perspective and removed Needs tech review Requires a review from the technical perspective labels Jul 20, 2026

@maximiliankolb maximiliankolb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@maximiliankolb maximiliankolb added style review done No issues from docs style/grammar perspective and removed Needs style review Requires a review from docs style/grammar perspective labels Jul 20, 2026

@ianballou ianballou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

stejskalleos pushed a commit to theforeman/foremanctl that referenced this pull request Jul 21, 2026
@aneta-petrova
aneta-petrova merged commit 6d9f674 into theforeman:master Jul 21, 2026
12 of 13 checks passed
@aneta-petrova
aneta-petrova deleted the rake-migrate-deb branch July 21, 2026 08:17
aneta-petrova added a commit that referenced this pull request Jul 21, 2026
(cherry picked from commit 6d9f674)
(cherry picked from commit 9d11677)
aneta-petrova added a commit that referenced this pull request Jul 21, 2026
aneta-petrova added a commit that referenced this pull request Jul 21, 2026
aneta-petrova added a commit that referenced this pull request Jul 21, 2026
aneta-petrova added a commit that referenced this pull request Jul 21, 2026
aneta-petrova added a commit that referenced this pull request Jul 21, 2026
@aneta-petrova

aneta-petrova commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

Merged to "master' and cherry-picked:

4d001dd..d891409 3.19 -> 3.19
4f15a6c..b5c23e1 3.18 -> 3.18
83efff7..f630618 3.17 -> 3.17
a3d7f58..e450155 3.16 -> 3.16
6a795a0..9d11677 3.15 -> 3.15 (a minor conflict)
3b59d78..d29fd61 3.14 -> 3.14 (picked from 3.15)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

style review done No issues from docs style/grammar perspective tech review done No issues from the technical perspective

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants