#22 Fix F24 atomic.
Merged by robyduck. Opened by ralph.
f24-atomic  into  master

Download 22.patch
  • First, I reverted a number of workaround commits used to get things working while atomic was broken.
  • Second, I fixed a few bugs in atomic_vars that resulted from changed image names and urls.
  • Third, I removed all the old old default AMIs in globalvars. They cause us to think things are working when they are really not.
  • I fixed some urls in the atomic.html template.
  • I got the AMI polling working again for atomic which needed a url-pattern change.

I can't verify the details on this myself offhand, but it looks right at the high level. Thank you so much for diving into this!

I applied the patch on a local checkout and I get three issues on the Cloud Atomic download page:

  1. There's a general warning banner that states: "The latest two week build did not meet our testing criteria. The images available are from over ??? days ago. Check the Project Atomic blog for updates and information about Atomic blocker bugs."
  2. All the sizes report "???MB," rather than something exact.
  3. The link to the 64-bit Atomic Host image in the right bar appears to be wrong ("Cloud_Atomic" vs. "Atomic" on the mirror I got at rit.edu).

Is this an issue with the globals somewhere, or something else missing? Something I'm failing to do in testing?

@pfrields, I think issues number 1 and 2 are just hiccups. Those work for me locally.

I think issue #3 is actually an issue though, but it is a pre-existing one. AFAIK, we've never known what to do with that rightmost bar. There may have been some plan for it that never bore fruit.

Yeah, here are my logs from the build process. @pfrields, do you need to make veryclean?

INFO:atomic_vars:Looking for latest atomic release for 24
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): apps.fedoraproject.org
INFO:atomic_vars: Found composedate: 20160712.0
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): dl.fedoraproject.org
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): dl.fedoraproject.org
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): dl.fedoraproject.org
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): dl.fedoraproject.org
INFO:fedimg_vars:Looking for AMIs for Fedora-Cloud-Base-24-20160616.n.0.x86_64
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): apps.fedoraproject.org
INFO:fedimg_vars:Looking for AMIs for Fedora-Atomic-24-20160712.0.x86_64
INFO:atomic_vars:Looking for latest atomic release for 24
INFO:atomic_vars: Found composedate: 20160712.0
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): dl.fedoraproject.org
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): dl.fedoraproject.org
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): dl.fedoraproject.org
INFO:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): dl.fedoraproject.org

The fixes are working fine, but we need to make sure the images are synched out to the mirrors. Also, we are missing the Atomic Iso installer, in alt/atomic/stable I can find only a Fedora-Atomic-dvd.

Fianlly, we need to revert also commit 5ae1792ee23e2a5c8ee745395211c6fde685e0d9 and 1fb82e9ee64b5008ca6fd99ea5b1a930b1e21364, dropping also the 32 bit checksum part.

All the rest looks good to me.

The fixes are working fine, but we need to make sure the images are synched out to the mirrors. Also, we are missing the Atomic Iso installer, in alt/atomic/stable I can find only a Fedora-Atomic-dvd.
Fianlly, we need to revert also commit 5ae1792ee23e2a5c8ee745395211c6fde685e0d9 and 1fb82e9ee64b5008ca6fd99ea5b1a930b1e21364, dropping also the 32 bit checksum part.
All the rest looks good to me.

When you say "we are missing the Atomic ISO installer" what are you referring to other than the Fedora-Atomic-DVD?

Yeah, here are my logs from the build process. @pfrields, do you need to make veryclean?

I think the conversation has moved on already so I don't mean to be a pain. I did try make veryclean but it didn't make a difference for me.

Pull-Request has been merged by robyduck

Metadata