Skip to content

Script to check obs links#73

Open
juliogonzalez wants to merge 1 commit intoSUSE:masterfrom
juliogonzalez:tools
Open

Script to check obs links#73
juliogonzalez wants to merge 1 commit intoSUSE:masterfrom
juliogonzalez:tools

Conversation

@juliogonzalez
Copy link
Copy Markdown
Member

@juliogonzalez juliogonzalez commented Apr 20, 2020

To be used for now to check that the salt links are OK, so we don't have this again: https://lists.opensuse.org/uyuni-users/2020-04/msg00016.html

This will be executed as part of a Jenkins job (most probably a pipeline, as long as I can see how to parametrize them)

$ ./check-obs-link -a https://api.suse.de -p Devel:Galaxy:Manager:3.2 -l openSUSE.org:systemsmanagement:saltstack:products -j salt; echo $?
[INFO ] Checking Devel:Galaxy:Manager:3.2/salt
[OK   ] The package links to openSUSE.org:systemsmanagement:saltstack:products/salt
0
$ ./check-obs-link -a https://api.suse.de -p Devel:Galaxy:Manager:4.0 -l openSUSE.org:systemsmanagement:saltstack:products -j salt; echo $?
[INFO ] Checking Devel:Galaxy:Manager:4.0/salt
[OK   ] The package links to openSUSE.org:systemsmanagement:saltstack:products/salt
0
$ ./check-obs-link -a https://api.suse.de -p Devel:Galaxy:Manager:Head -l openSUSE.org:systemsmanagement:saltstack:products -j salt; echo $?
[INFO ] Checking Devel:Galaxy:Manager:Head/salt
[ERROR] The package links to openSUSE.org:systemsmanagement:saltstack:products:next/salt
1
$ ./check-obs-link -a https://api.opensuse.org -p systemsmanagement:Uyuni:Master -l openSUSE.org:systemsmanagement:saltstack:products -j salt; echo $?
[INFO ] Checking systemsmanagement:Uyuni:Master/salt
[ERROR] The package links to systemsmanagement:saltstack:products/salt
1

@srbarrios
Copy link
Copy Markdown
Member

srbarrios commented Apr 23, 2020

Looks good to me, but I'm wondering if we should move this to somewhere available in uyuni-project git organization, could it make sense? If it don't, fine to have that tools folder here.

@moio
Copy link
Copy Markdown
Member

moio commented Sep 9, 2020

@juliogonzalez can this be merged?

@juliogonzalez
Copy link
Copy Markdown
Member Author

Ugh... I lost track of this. The script itself is ready to get merged, but since I proposed pipelines, let's not merge PR and let's create the pipelines right here (since we have all the pipelines in this place).

@srbarrios srbarrios force-pushed the master branch 30 times, most recently from 6084e40 to 3fdeb84 Compare March 1, 2022 16:07
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.

3 participants