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.

Sign up for GitHub

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

Jump to bottom

[master] Refresh repositories with changed URL and reload them again #1100

Merged
merged 9 commits into from
Nov 1, 2023
Merged

[master] Refresh repositories with changed URL and reload them again #1100

merged 9 commits into from
Nov 1, 2023

Conversation

lslezak
Copy link
Member

@lslezak lslezak commented Nov 1, 2023

jreidinger and others added 9 commits September 1, 2023 22:14
@jreidinger
Bump version to 4.6.0
8af4d14
@jreidinger
Merge commit '256c69b' into backport_sp6
5992623
@jreidinger
Merge pull request #1097 from yast/backport_sp6
7b057a9 
Backport sp6
@lslezak
Refresh repositories with changed URL and reload them again
76abd3e 
...to activate the changes (related to bsc#1215884)
- 4.5.18
@lslezak
Merge pull request #1098 from yast/sp5_fix_old_repo_refresh
482ef3d 
[SLE15-SP5] Refresh repositories with changed URL and reload them again
@lslezak
Refresh repositories with changed URL (related to bsc#1215884)
d856536 
...and reload them again to activate the changes

(merged a change from the SP5 branch)
@lslezak
Merge pull request #1099 from yast/sp6_fix_old_repo_refresh
01c2af1 
[SLE15-SP6] Refresh repositories with changed URL and reload them again
@lslezak
Merge remote-tracking branch 'origin/SLE-15-SP6' into tw_fix_old_repo…
5a8ff1f 
…_refresh
@lslezak
Use new Y2Packager::Repository.refresh method
1fa02d2
mvidner
mvidner approved these changes Nov 1, 2023
View reviewed changes
Copy link
Member

@mvidner mvidner left a comment

Choose a reason for hiding this comment

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

LGTM

The CI failure looks triggered only by the new refresh method which I dare say is correct

@coveralls
Copy link

coveralls commented Nov 1, 2023 •
edited
Loading

Coverage Status

coverage: 40.845% (+0.04%) from 40.807% when pulling 1fa02d2 on tw_fix_old_repo_refresh into 7ec50d3 on master.

@lslezak
Copy link
Member Author

lslezak commented Nov 1, 2023

Restarting the CI after merging the other PR fixed the problems.

@lslezak lslezak merged commit 8c9bd4c into master Nov 1, 2023
14 checks passed
@lslezak lslezak deleted the tw_fix_old_repo_refresh branch November 1, 2023 12:05
@yast-bot
Copy link
Contributor

yast-bot commented Nov 1, 2023

✔️ Internal Jenkins job #149 successfully finished
✔️ Created OBS submit request #1121603

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

@mvidner mvidner mvidner approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lslezak @coveralls @yast-bot @mvidner @jreidinger