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

fix: Fresh installs must get the correct URL for rh-certified #2078

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

rochacbruno
Copy link
Member

Add /content/published to the default URL when setting up the default remotes.

This has been fix on #2032 but that PR not backported.

This PR changes the original migration, so fresh installs gets the new URL and existing systems are not affected

This must be backported to 4.9.z

No-Issue
Related: AAH-2836

@github-actions github-actions bot added backport-4.2 This PR should be backported to stable-4.2 (1.2) backport-4.4 This PR should be backported to stable-4.4 (2.1) backport-4.5 This PR should be backported to stable-4.5 (2.2) backport-4.6 This PR should be backported to stable-4.6 (2.3) backport-4.7 This PR should be backported to stable-4.7 (2.4) backport-4.8 This PR should be backported to stable-4.8 (2.4) backport-4.9 This PR should be backported to stable-4.9 (2.4) labels Feb 21, 2024
@rochacbruno rochacbruno removed backport-4.2 This PR should be backported to stable-4.2 (1.2) backport-4.4 This PR should be backported to stable-4.4 (2.1) backport-4.5 This PR should be backported to stable-4.5 (2.2) backport-4.6 This PR should be backported to stable-4.6 (2.3) backport-4.7 This PR should be backported to stable-4.7 (2.4) backport-4.8 This PR should be backported to stable-4.8 (2.4) labels Feb 21, 2024
@rochacbruno rochacbruno marked this pull request as ready for review February 21, 2024 18:42
@newswangerd newswangerd self-requested a review February 21, 2024 19:09
Add /content/published to the default URL when setting up the
default remotes.

This has been fix on ansible#2032 but that PR not backported.

This PR changes the original migration, so fresh installs gets the new URL
and existing systems are not affected

This must be backported to 4.9.z

No-Issue
Related: AAH-2836
@rochacbruno rochacbruno force-pushed the fix_remote_url_for_console branch from 56f0b36 to 001739e Compare February 29, 2024 17:35
@rochacbruno rochacbruno merged commit 5d56bdd into ansible:master Feb 29, 2024
27 of 28 checks passed
@rochacbruno rochacbruno deleted the fix_remote_url_for_console branch February 29, 2024 18:53
Copy link

patchback bot commented Feb 29, 2024

Backport to stable-4.9: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.9/5d56bddb667762693df7d3e2bb18ab4b768d25af/pr-2078

Backported as #2089

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Feb 29, 2024
Add /content/published to the default URL when setting up the
default remotes.

This has been fix on #2032 but that PR not backported.

This PR changes the original migration, so fresh installs gets the new URL
and existing systems are not affected

This must be backported to 4.9.z

No-Issue
Related: AAH-2836

(cherry picked from commit 5d56bdd)
rochacbruno added a commit that referenced this pull request Mar 1, 2024
… the correct URL for rh-certified (#2089)

fix: Fresh installs must get the correct URL for rh-certified (#2078)

Add /content/published to the default URL when setting up the
default remotes.

This has been fix on #2032 but that PR not backported.

This PR changes the original migration, so fresh installs gets the new URL
and existing systems are not affected

This must be backported to 4.9.z

No-Issue
Related: AAH-2836

(cherry picked from commit 5d56bdd)

Co-authored-by: Bruno Rocha <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4.9 This PR should be backported to stable-4.9 (2.4)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants