1
0
mirror of https://pagure.io/fedora-qa/createhdds.git synced 2025-07-19 18:11:34 +00:00

Compare commits

...

3 Commits

Author SHA1 Message Date
Adam Williamson
b44312dfe3 Reduce longer timeout for aarch64
Well, the longer timeout isn't really helping. The aarch64 builds
just fail a lot and I don't really know why. Drop it to 4500, as
I really did see it take 4000 seconds on one successful manual
attempt...

Signed-off-by: Adam Williamson <awilliam@redhat.com>
2024-03-29 08:28:45 -07:00
Adam Williamson
f96e2991fe Extend virt-install timeout on aarch64
I think aarch64 desktop images are often failing to build because
it takes longer than the timeout, so let's bump it a bit.

Signed-off-by: Adam Williamson <awilliam@redhat.com>
2024-03-24 09:00:24 -07:00
Adam Williamson
81e1a18c27 Drop handling of obsolete arches, clean up some explanations
Signed-off-by: Adam Williamson <awilliam@redhat.com>
2024-03-24 08:59:02 -07:00

View File

@ -63,9 +63,9 @@ def supported_arches():
"""Provides a list of the arches for which virt-install images can """Provides a list of the arches for which virt-install images can
be built on this host. be built on this host.
""" """
powerpc_arches = ['ppc64', 'ppc64le', 'noarch'] powerpc_arches = ['ppc64le', 'noarch']
intel_arches = ['i686', 'x86_64', 'noarch'] intel_arches = ['x86_64', 'noarch']
aarch64_arches = ['aarch64', 'armv7l'] aarch64_arches = ['aarch64']
if CPUARCH in powerpc_arches: if CPUARCH in powerpc_arches:
supported_arches = powerpc_arches supported_arches = powerpc_arches
elif CPUARCH in intel_arches: elif CPUARCH in intel_arches:
@ -294,14 +294,14 @@ class VirtInstallImage(object):
arch = self.arch arch = self.arch
fedoradir = 'fedora/linux' fedoradir = 'fedora/linux'
memsize = '3072' memsize = '3072'
if arch == 'i686': timeout = 3600
arch = 'i386' if arch == "ppc64le":
if arch in ['ppc64','ppc64le']:
fedoradir = 'fedora-secondary' fedoradir = 'fedora-secondary'
memsize = '4096' memsize = '4096'
if arch == 'i386': if arch == "aarch64":
# i686 is in fedora-secondary (until it died) # desktop images take quite a while to build on aarch64,
fedoradir = 'fedora-secondary' # particularly on the slower worker on stg
timeout = 4500
variant = self.variant variant = self.variant
# From F31 onwards, Workstation tree is not installable and we # From F31 onwards, Workstation tree is not installable and we
@ -352,7 +352,7 @@ class VirtInstallImage(object):
logger.debug("Command: %s", ' '.join(args)) logger.debug("Command: %s", ' '.join(args))
if not textinst: if not textinst:
logger.info("Connect via VNC to monitor") logger.info("Connect via VNC to monitor")
ret = subprocess.call(args, timeout=3600) ret = subprocess.call(args, timeout=timeout)
except subprocess.TimeoutExpired: except subprocess.TimeoutExpired:
logger.warning("Image creation timed out!") logger.warning("Image creation timed out!")
# clean up the domain again # clean up the domain again
@ -827,23 +827,23 @@ def parse_args(hdds):
imgparser.set_defaults(imggrp=('guestfs', imggrp)) imgparser.set_defaults(imggrp=('guestfs', imggrp))
# For virtinstall images, we provide args to override the release/ # For virtinstall images, we provide args to override the release/
# arch combination; using args.release will always result in just a # arch combination; using args.release without args.arch will always
# single image being built, for x86_64 unless args.arch is set to # result in just a single image being built, for x86_64; using
# i686. # args.arch without args.release is ignored
for imggrp in hdds['virtinstall']: for imggrp in hdds['virtinstall']:
imgparser = subparsers.add_parser( imgparser = subparsers.add_parser(
imggrp['name'], description="Create {0} image(s)".format(imggrp['name'])) imggrp['name'], description="Create {0} image(s)".format(imggrp['name']))
imgparser.add_argument( imgparser.add_argument(
'-r', '--release', help="The release to build the image(s) for. If not " '-r', '--release', help="The release to build the image(s) for. If not "
"set, createhdds will attempt to determine the current release and build " "set, createhdds will attempt to determine the current release and build "
"for appropriate releases relative to that", "for appropriate releases relative to that, and will ignore --arch",
default='') default='')
imgparser.add_argument( imgparser.add_argument(
'-a', '--arch', help="The arch to build the image(s) for. If neither " '-a', '--arch', help="The arch to build the image(s) for. Must be set "
"this nor --release is set, createhdds will decide the appropriate " "with --release or else it is ignored. If neither this nor --release is set, "
"arch(es) to build for each release. If this is not set but --release " "createhdds will decide the appropriate arch(es) to build for each release. "
"is set, only x86_64 image(s) will be built.", "If this is not set but --release is set, only x86_64 image(s) will be built.",
choices=('x86_64', 'i686', 'ppc64le', 'aarch64')) choices=('x86_64', 'ppc64le', 'aarch64'))
imgparser.set_defaults(func=cli_image) imgparser.set_defaults(func=cli_image)
# Here we're stuffing the type of the image and the dict from # Here we're stuffing the type of the image and the dict from
# hdds into args for cli_image() to use. # hdds into args for cli_image() to use.