-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
docs: update dnf config-manager
usage
#35980
Conversation
In Fedora Linux 41, which uses DNF5, backward compatibility with DNF4 was intentionally broken. The method for adding repositories via dnf config-manager has changed to use new sub-commands, such as addrepo. Signed-off-by: Bruno Schaatsbergen [email protected]
This is a fair point but I think we need to consider the wider perspective, which is that this is only one of many places where such docs exist. https://developer.hashicorp.com/terraform/cli/install/yum I'm not even sure where exactly do we link from to that page. What is IMHO more impactful and visible place which will certainly need updating is https://developer.hashicorp.com/terraform/install which I think is using a shared component between all product sites. All that is to say, given that the installation instructions are pretty much the same for all products I'm not sure there is much value in maintaining dedicated (Terraform specific) pages like these at all? cc @hashicorp/terraform-education Any opinions or ideas on this topic? |
Thanks for tagging us into this interesting one! ✨
So this page is live, but we currently hide it in the sidenav, so folks can only find it by entering the URL. I think we can honestly just remove the page at this point, and redirect to the main Terraform installation docs as you suggested, Radek. I'll follow-up with my team on how we can update this guidance in the main Terraform installation docs (https://developer.hashicorp.com/terraform/install). However, I do think we should merge this to fix the Stacks CLI article. While Stacks are in public beta we are grouping all the pages together including the TFStacks CLI reference and installation page. Once we GA this will live somewhere else, but for now we should use Bruno's suggestion to update it where it is if that helps folks. |
@rkoron007 Sounds good to me, as long as you keep track of this somewhere. |
Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch. |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Closes #35979
In Fedora Linux 41 which uses DNF5, backward compatibility with DNF4 was intentionally broken. The
dnf config-manager
now uses new sub-commands, likeaddrepo
, for adding repositories.Since F41 is the latest supported Fedora Linux release, this PR proposes updating the instructions to reflect DNF5 installation.