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

Add new utility script to be used to create the fedora-install-X repositories #58

Merged
merged 7 commits into from
May 4, 2015
Merged

Add new utility script to be used to create the fedora-install-X repositories #58

merged 7 commits into from
May 4, 2015

Conversation

pypingou
Copy link
Member

No description provided.

@ralphbean
Copy link
Contributor

This looks okay to me, but it's one of those things that will just need to be tested before we can say anything certain about it.

@pypingou
Copy link
Member Author

Seems to run fine like this in stg, but not sure how to check it

@adrianreber
Copy link
Member

To test it you need to copy the files from /var/lib/mirrormanager (after they have re-created by mm2_update-mirrorlist-server) to the mirrorlist server and kill -1 the mirrorlistserver process

@pypingou
Copy link
Member Author

hm, I'm not sure I follow you, it's a one off script taking data from the disk and adding it to the DB.

To run it, on mm-backend01.stg, I have been using:
mm2_create_install_repo --version=21 --category='Fedora Linux' /srv/pub/fedora/linux/releases/21/
which seemed logical and seems to have added things in the DB: https://admin.stg.fedoraproject.org/mirrormanager2/admin/repositoryview/?page=57 but apparently there is still something missing

@pypingou
Copy link
Member Author

Ok I think I got it: https://admin.stg.fedoraproject.org/mirrormanager2/admin/repositoryview/?sort=5&page=15

@pypingou
Copy link
Member Author

I ran it with
cd /srv/ && mm2_create_install_repo --version=21 --category='Fedora Linux' pub/fedora/linux/releases/21/

@pypingou
Allow to update a directory that did not exist
8a222c3 
Sometime when we run the script there is an error and that leads to the
data in the DB to be incomplete (for example: missing a Directory), with
this change, we can re-run the script and we will update what was wrong
fine.
@pypingou
Enforce providing at least one positional argument to the script and …
94d4199 
…give an example
@mdomsch
Copy link
Contributor

mdomsch commented May 2, 2015

Why is that not done in umdl like the other repos?
On Apr 30, 2015 2:50 PM, "Pierre-Yves Chibon" notifications@github.com
wrote:

I ran it with

cd /srv/ && mm2_create_install_repo --version=21 --category='Fedora Linux'
pub/fedora/linux/releases/21/

—
Reply to this email directly or view it on GitHub
#58 (comment)
.

@adrianreber
Copy link
Member

@mdomsch: I guess this is 'historically' (as always). It could be part of umdl but it needs different paths starting with Fedora 21:

fedora/linux/releases/21/Server/x86_64/os/
fedora/linux/releases/20/Fedora/x86_64/os/

This is probably worth a future umdl feature. I will open a ticket to integrate it into umdl.

@pypingou
Copy link
Member Author

pypingou commented May 4, 2015

So let's merge this for now so that we can cut a new release :)

pypingou added a commit that referenced this pull request May 4, 2015
@pypingou
Merge pull request #58 from fedora-infra/create_repo
4e1298f 
Add new utility script to be used to create the fedora-install-X repositories
@pypingou pypingou merged commit 4e1298f into master May 4, 2015
@pypingou pypingou deleted the create_repo branch May 4, 2015 10:46
@adrianreber adrianreber mentioned this pull request May 14, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@pypingou @ralphbean @adrianreber @mdomsch