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

Add info about refreshing software repositories #584

Merged
merged 6 commits into from
Oct 14, 2020

Conversation

keichwa
Copy link
Contributor

@keichwa keichwa commented Oct 1, 2020

uyuni-project/uyuni#2645
https://github.com/SUSE/spacewalk/issues/12612

Description

Short summary of why you created this PR

Target branches

Which product version does this PR apply to?

  • Manager-4.0
  • Manager-4.1
  • Manager-4.2
  • Uyuni

Links

Fixes #<insert issue or PR link, if any>

@keichwa
Copy link
Contributor Author

keichwa commented Oct 1, 2020

@juliogonzalez or @mcalmer can you please confirm that we actually need a strategic zypper ref here?

@mcalmer
Copy link
Contributor

mcalmer commented Oct 1, 2020

I would say "no". At least 'zypper patch' should do it automatically and I think 'zypper list-patches' should do it as well.
But a refresh happens only when the cache is declared as outdated. I do not know about the exact time. Maybe 5 Minutes, but could also be 24 hours.
Please ask you zypp guys about the rules.

@juliogonzalez
Copy link
Member

All I can tell is that I always manually run zypper ref before running zypper in, zypper up, etc... Just in case.

But could be because I am a long time Debian user, where we need to refresh the metadata :-)

So I agree with @mcalmer, check with zypp guys.

@mlandres
Copy link

mlandres commented Oct 1, 2020

It depends on how autorefresh is configured for the repos!

  • zypper ref checks and refreshes all repositories.
  • zypper CMD checks and refreshes autorefresh=1 repositories.

So if you know all enabled repos have autorefresh on, you don't need an explicit refresh. Otherwise you do (unless you know the repos do not contribute to the command).

@mlandres
Copy link

mlandres commented Oct 1, 2020

I do not know about the exact time. Maybe 5 Minutes, but could also be 24 hours.

The default no-repo-refresh-again timeout is /etc/zypp/zypp.conf:repo.refresh.delay = 10 min.
(Future zypp versions may allow a value per repo).

@keichwa
Copy link
Contributor Author

keichwa commented Oct 1, 2020 via email

@keichwa
Copy link
Contributor Author

keichwa commented Oct 5, 2020

I added a note and consider it ready for review.

Copy link
Contributor

@Loquacity Loquacity left a comment

Choose a reason for hiding this comment

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

Just some suggestions to reduce the verbiage.

modules/upgrade/pages/server-minor-upgrade-uyuni.adoc Outdated Show resolved Hide resolved
modules/upgrade/pages/server-minor-upgrade-uyuni.adoc Outdated Show resolved Hide resolved
@keichwa
Copy link
Contributor Author

keichwa commented Oct 6, 2020 via email

keichwa and others added 2 commits October 6, 2020 06:40
Co-authored-by: Lana Brindley <lbrindley@suse.de>
Co-authored-by: Lana Brindley <lbrindley@suse.de>
@keichwa
Copy link
Contributor Author

keichwa commented Oct 6, 2020

@Loquacity, @mcalmer, and @juliogonzalez , and now the question is whether we want to add this note everywhere where we have this or a similar procedure?

@Loquacity
Copy link
Contributor

@Loquacity, @mcalmer, and @juliogonzalez , and now the question is whether we want to add this note everywhere where we have this or a similar procedure?

I think that could easily get out of hand. Let's save it for places where we know it's confusing.

@keichwa
Copy link
Contributor Author

keichwa commented Oct 6, 2020 via email

@mcalmer
Copy link
Contributor

mcalmer commented Oct 6, 2020

How many places do we have where we describe to call zypper cli directly?
I think this is true for ther server connected to SCC directly. All other systems are managed by the server and that procedure is not needed.

I think when you just find one more place, just duplicate it. If it is more, we might want to check if these descriptions are correct and needed at all.

@keichwa
Copy link
Contributor Author

keichwa commented Oct 6, 2020

I think when you just find one more place, just duplicate it. If it is more, we might want to check if these descriptions are correct and needed at all.

You are right. With grep, I find it just a 2nd time. I guess I'll duplicate it.

@Loquacity
Copy link
Contributor

It was the "or similar procedure" that made me think it would get out of hand 😉

@keichwa keichwa merged commit df20447 into master Oct 14, 2020
@keichwa keichwa deleted the uyuni-server-upgrade--#2645-keichwa branch October 14, 2020 07:50
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.

None yet

5 participants