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

Improve how docs describe adding repo to SLES #9532

Closed
wants to merge 1 commit into from

Conversation

htriem
Copy link
Contributor

@htriem htriem commented Sep 29, 2022

This contains a more efficient way of adding the Icinga repos to a SLES system. For more information, check out ticket number 42655.

@cla-bot cla-bot bot added the cla/signed label Sep 29, 2022
@htriem htriem force-pushed the docs/add-sles-repo-via-curl/42655 branch from 728a80f to ea93486 Compare October 4, 2022 13:47
@julianbrost
Copy link
Contributor

ref/IP/42655

```bash
rpm --import https://packages.icinga.com/icinga.key

zypper ar https://packages.icinga.com/subscription/sles/ICINGA-release.repo
zypper ar -r https://packages.icinga.com/subscription/sles/$releasever/release/ icinga-stable-release
Copy link
Contributor

Choose a reason for hiding this comment

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

Where does the variable $releasever come from?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well, I found this while looking for the answer: https://doc.opensuse.org/projects/libzypp/HEAD/zypp-repovars.html

It seems like every SuSE-related version uses this variable to determine the version of the OS.

Copy link
Member

Choose a reason for hiding this comment

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

Yes, if it would be quoted. Have you tried to provide credentials?

Copy link
Member

Choose a reason for hiding this comment

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

@htriem PING

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've provided them, still throws some errors. I've decided on a different way of adding the repos more in line with the RHEL way.

@htriem htriem force-pushed the docs/add-sles-repo-via-curl/42655 branch from ea93486 to 14959cb Compare January 9, 2023 00:45
@htriem
Copy link
Contributor Author

htriem commented Jan 9, 2023

After some testing, I've changed the method to use wget, which is provided with SLES and more in line with the way we add repos under RHEL. Please check if this is more statisfactory.

Also, I've edited the related PRs as well.

@julianbrost
Copy link
Contributor

What's wrong with doing the exact suggestion that was made in the ticket, i.e. just removing the {username}:{password}@ placeholders from the .repo file altogether?

Then the existing instructions work as-is an zypper asks for the credentials as needed:

0de8cd4736f8:/ # zypper ar https://gist.githubusercontent.com/julianbrost/5f90e5df34ee83a1bfd94cf545be9a6b/raw/af28f08635a29cbd97cee21a61b61b9e0fc1496c/ICINGA-release.repo
Adding repository 'ICINGA (stable release for SLES 15.4)' ................[done]
Repository 'ICINGA (stable release for SLES 15.4)' successfully added

URI         : https://packages.icinga.com/subscription/sles/15.4/release/
Enabled     : Yes
GPG Check   : Yes
Autorefresh : No
Priority    : 99 (default priority)

Repository priorities in effect:                (See 'zypper lr -P' for details)
      99 (default priority) : 36 repositories
     100 (lowered priority) :  1 repository
0de8cd4736f8:/ # zypper ref
[...]
Authentication required for 'https://packages.icinga.com/subscription/sles/15.4/release/'
User Name: [...]
Password: [...]
Retrieving repository 'ICINGA (stable release for SLES 15.4)' metadata ...[done]
Building repository 'ICINGA (stable release for SLES 15.4)' cache ........[done]
All repositories have been refreshed.

@htriem
Copy link
Contributor Author

htriem commented Mar 7, 2023

The Username and Password section have been removed from the .repo file, so there is no need for this PR anymore.

@htriem htriem closed this Mar 7, 2023
@icinga-probot icinga-probot bot deleted the docs/add-sles-repo-via-curl/42655 branch March 7, 2023 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants