#271 atomichost: Use prod ref, not /updates
Merged 3 years ago by mohanboddu. Opened 3 years ago by walters.
walters/fedora-kickstarts f26-ah-updates  into  f26

atomichost: Use prod ref, not /updates
Colin Walters • 3 years ago  
file modified
+12 -7
@@ -30,18 +30,23 @@ 

  volgroup atomicos pv.01

  logvol / --size=3000 --fstype="xfs" --name=root --vgname=atomicos


- # Equivalent of %include fedora-repo.ks

+ # Equivalent of %include fedora-repo.ks; note this uses the Bodhi ref, which we reset below

  ostreesetup --nogpg --osname=fedora-atomic --remote=fedora-atomic --url=https://kojipkgs.fedoraproject.org/atomic/26/ --ref=fedora/26/x86_64/updates/atomic-host




  %post --erroronfail

- # See https://github.com/projectatomic/rpm-ostree/issues/42

- # we only need this for before f26 release where we are using a

- # temporary "dev" repo url above. We want to change the remote

- # we use to match where the repo will live for the entirety of f26.

- ostree remote delete fedora-atomic

- ostree remote add --set=gpg-verify=true --set=gpgkeypath=/etc/pki/rpm-gpg/RPM-GPG-KEY-fedora-26-primary fedora-atomic 'https://kojipkgs.fedoraproject.org/atomic/26/'

+ # Set the origin to the "main ref", distinct from /updates/ which is where bodhi writes.

+ # We want consumers of this image to track the two week releases.

+ ostree admin set-origin --index 0 fedora-atomic https://kojipkgs.fedoraproject.org/atomic/26/ fedora/26/x86_64/atomic-host


+ # Make sure the ref we're supposedly sitting on (according

+ # to the updated origin) exists.

+ ostree refs fedora-atomic:fedora/26/x86_64/updates/atomic-host --create fedora-atomic:fedora/26/x86_64/atomic-host


+ # Remove the old ref so that the commit eventually gets

+ # cleaned up.

+ ostree refs fedora-atomic:fedora/26/x86_64/updates/atomic-host --delete


  # older versions of livecd-tools do not follow "rootpw --lock" line above

  # https://bugzilla.redhat.com/show_bug.cgi?id=964299

We need the same bits that live in the f25 branch now.

I also deleted the remote add/delete, since I think they're
no longer necessary.

thanks - pulling down the patch and testing it now

works - LGTM - please merge

I don't have merge access AFAICS.

Isn't it fedora-atomic/26/x86_64/atomic-host ?

agree - let's update the comment (even though it is cosmetic)

no - we changed out ref to be fedora/26/x86_64/atomic-host


3 years ago

Pull-Request has been merged by mohanboddu

3 years ago

rebase looks good to me