~rcj/ubuntu-on-ec2/ec2-publishing-scripts : changes

~rcj/ubuntu-on-ec2/ec2-publishing-scripts » Changes from revision 667

From Revision 667 to 648

Rev	Summary	Authors	Date
667	run-instances: Extend associate-ip timeout and correct retry... run-instances: Extend associate-ip timeout and correct retry logic The timeout is proving too short in production. This patch extends the timeout from 30s to 60s. Additionally the logic was running the command to retry before evaluating the loop counter and then the code would evaluate the retry loop counter after the loop to determine failure. This allows for one last execution of the command (xc2 associate-address) on the 6th time through the while loop but the while loop does not continue because the other condition for the while '[ $n -lt 5]' would be false. It is possible (and was shown in production) that the command was succeeding but only on the 6th attempt. The code follow the while loop looked to see that the loop had been run 6 times and automatically failed this despite a success. This patch changes the while evaluation to only look at the counter conditional before the execution of the program in the body of the while loop where success will break out from the loop early. This should be much cleaner and easier to spot issues.	Robert C Jennings	5 years ago
666	Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-que... Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-querydata-arch into lp:~ubuntu-on-ec2/ubuntu-on-ec2/ec2-publishing-scripts [a=philroche] [r=daniel-thewatkins,rcj] Bugfix: Update querydata merge code to include arch in query data output filenames This was added in aws publication code in awslib, which is called by cloud-image-manager, to enable multiple different architectures to output query data, which will be merged together later, but in this instance we are only interested in amd64 (hvm is amd64 only) so we can hardcode the arch to amd64. MP: https://code.launchpad.net/~philroche/ubuntu-on-ec2/ec2-publishing-scripts-querydata-arch/+merge/361829	Phil Roche	5 years ago
665	Add disco to ubuntu-adj2version [a=fginther] [r=philroche] Add disco to ubuntu-adj2version [a=fginther] [r=philroche] MP: https://code.launchpad.net/~fginther/ubuntu-on-ec2/ec2-publishing-scripts-disco/+merge/358636	Francis Ginther	5 years ago
664	Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-ker... Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-kernel-panic-revert into lp:~ubuntu-on-ec2/ubuntu-on-ec2/ec2-publishing-scripts [a=philroche] [r=daniel-thewatkins,rcj] Revert temporary use of previous Trusty AMIs for publication MP: https://code.launchpad.net/~philroche/ubuntu-on-ec2/ec2-publishing-scripts-kernel-panic-revert/+merge/353426	Phil Roche	5 years ago
663	Temporarily use an previous AMI for publication due to 20180... Temporarily use an previous AMI for publication due to 20180814 Trusty kernel panic	Phil Roche	5 years ago
662	Merge lp:~rcj/ubuntu-on-ec2/ec2-publishing-scripts into lp:~... Merge lp:~rcj/ubuntu-on-ec2/ec2-publishing-scripts into lp:~ubuntu-on-ec2/ubuntu-on-ec2/ec2-publishing-scripts [a=rcj] [r=daniel-thewatkins,philroche] ubuntu-adj2version: Explicit release definition Since we've looped in the alphabet and are approaching dd-series this code would have returned 6.06 for our next release rather than 19.04. The safest way to do this is by explicit suite matching to release version. MP: https://code.launchpad.net/~rcj/ubuntu-on-ec2/ec2-publishing-scripts/+merge/345373	Robert C Jennings	5 years ago
661	Drop use of uefi1 images in Xenial Drop use of uefi1 images in Xenial The use of uefi1 was limited to HVM:instance-store, although the comment states "HVM" this is incorrect. This results in an unnecessary image delta on Xenial for one of the four image types published. By reverting this we have a unified set of images on Xenial. Presently this helps with a uefi1 base image issue that only affects ability to boot on EC2 and sets the stage for a future migration off ec2-publishing-scripts entirely where all images are published from a common disk format. Trusty will still have this single uefi1 image in it's matrix. [a=rcj][r=philroche,fginther] MP: https://code.launchpad.net/~rcj/ubuntu-on-ec2/ec2-publishing-scripts/+merge/340039	Robert C Jennings	6 years ago
660	Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-ima... Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-image-id-debug into lp:~ubuntu-on-ec2/ubuntu-on-ec2/ec2-publishing-scripts [a=philroche] [r=daniel-thewatkins,fginther] Add debug output to debug image id race condition MP: https://code.launchpad.net/~philroche/ubuntu-on-ec2/ec2-publishing-scripts-image-id-debug/+merge/337174	Phil Roche	6 years ago
659	Add retries for common failure paths Add retries for common failure paths [a=rcj][r=fginther,philroche] MP: https://code.launchpad.net/~rcj/ubuntu-on-ec2/ec2-publishing-scripts/+merge/336555	Robert C Jennings	6 years ago
658	Make AWS profile/partition used by CIM configurable [a=Odd_B... Make AWS profile/partition used by CIM configurable [a=Odd_Bloke][r=fginther,rcj] MP: https://code.launchpad.net/~daniel-thewatkins/ubuntu-on-ec2/ec2-publishing-scripts/+merge/334987	Daniel Watkins	6 years ago
657	Add bionic to ubuntu-adj2version [a=Odd_Bloke][r=philroche,t... Add bionic to ubuntu-adj2version [a=Odd_Bloke][r=philroche,tribaal] https://code.launchpad.net/~daniel-thewatkins/ubuntu-on-ec2/ec2-publishing-scripts/+merge/333544	Daniel Watkins	6 years ago
656	cloud-image-manager image registrations should not make publ... cloud-image-manager image registrations should not make public Instead force query data to be generated without making public.	Phil Roche	6 years ago
655	Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-cim... Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-cim-public into lp:~ubuntu-on-ec2/ubuntu-on-ec2/ec2-publishing-scripts [a=philroche] [r=fginther,rcj] When registering image with cloud-image-manager we need to make them public so get query data This reverts the previous change not making image public. If image is not public then no query data will be generated. This means we have no way of retrieving the AMI id registered. Making image public fixes that. MP: https://code.launchpad.net/~philroche/ubuntu-on-ec2/ec2-publishing-scripts-cim-public/+merge/328217	Phil Roche	6 years ago
654	Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-pro... Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-promote-disable-reuse into lp:~ubuntu-on-ec2/ubuntu-on-ec2/ec2-publishing-scripts [a=philroche] [r=daniel-thewatkins,rcj] Do not use reuse-ebs-from as it doesn't use cloud-image-manager Which we need to set attributes. This change is as a result of HVM EBS amis not having the required ENASupport attribute set. Bypassing reuse of snapshots will result in a longer promotion MP: https://code.launchpad.net/~philroche/ubuntu-on-ec2/ec2-publishing-scripts-promote-disable-reuse/+merge/328184	Phil Roche	6 years ago
653	Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-inc... Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-incorrect-release-label into lp:~ubuntu-on-ec2/ubuntu-on-ec2/ec2-publishing-scripts [a=philroche] [r=daniel-thewatkins,rcj] Registration or release image was using the incorrect label for query data This resulted in failed promotion and missing Trusty release AMIs. This commit uses the correct label. MP: https://code.launchpad.net/~philroche/ubuntu-on-ec2/ec2-publishing-scripts-incorrect-release-label/+merge/328160	Phil Roche	6 years ago
652	Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-cim... Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-cim-no-public into lp:~ubuntu-on-ec2/ubuntu-on-ec2/ec2-publishing-scripts [a=philroche] [r=fginther,rcj] Do not make images public if registered using CIM publicize-build should be making them public MP: https://code.launchpad.net/~philroche/ubuntu-on-ec2/ec2-publishing-scripts-cim-no-public/+merge/328172	Phil Roche	6 years ago
651	Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-ena... Merge lp:~philroche/ubuntu-on-ec2/ec2-publishing-scripts-ena-clean into lp:~ubuntu-on-ec2/ubuntu-on-ec2/ec2-publishing-scripts [a=philroche] [r=daniel-thewatkins,fginther,rcj] Modify Trusty old world publication to make it possible to set the ENA attribute using CIM Using CIM only for HVM image publication. MP: https://code.launchpad.net/~philroche/ubuntu-on-ec2/ec2-publishing-scripts-ena-clean/+merge/324937	Phil Roche	6 years ago
650	Add artful special-case to ubuntu-adj2version [a=Odd_Bloke][... Add artful special-case to ubuntu-adj2version [a=Odd_Bloke][r=fginther,rcj] https://code.launchpad.net/~daniel-thewatkins/ubuntu-on-ec2/ec2-publishing-scripts/+merge/322986	Daniel Watkins	7 years ago
649	[r=rcj] Use download query data to generate .latest.txt file... [r=rcj] Use download query data to generate .latest.txt files	Daniel Watkins	7 years ago
648	Don't write download query data for xenial Don't write download query data for xenial Back-port of EC2/Download dis-entanglement from yakkety to xenial	Robert C Jennings	7 years ago

Older »