Skip to content

Commit

Permalink
Merge pull request #5921 from clebergnu/pre_release_job_92_2
Browse files Browse the repository at this point in the history
Signed-off-by: Jan Richter <jarichte@redhat.com>
  • Loading branch information
richtja committed Apr 30, 2024
2 parents 685975e + f594dbc commit 640e856
Show file tree
Hide file tree
Showing 5 changed files with 168 additions and 88 deletions.
38 changes: 20 additions & 18 deletions avocado/utils/vmimage.py
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ def get_image_url(self):
else:
cloud = 'CloudImages'

if self.url_old_images and int(self.version) <= 34:
if self.url_old_images and int(self.version) <= 36:
self.url_versions = self.url_old_images

self.url_images = self.url_versions + '{version}/' \
Expand Down Expand Up @@ -290,11 +290,22 @@ def __init__(self, version='[0-9]+.[0-9]+', build=None,
self.url_images = self.url_versions + 'releases/{version}/release/'
self.image_pattern = 'ubuntu-(?P<version>{version})-server-cloudimg-(?P<arch>{arch}).img'

def get_best_version(self, versions):
""" Return best (more recent) version """
max_float = max([float(item) for item in versions])
return str(f'{max_float:2.2f}')

def get_versions(self):
"""Return all available versions for the current parameters."""
parser = VMImageHtmlParser(self.version_pattern)
self._feed_html_parser(self.url_versions, parser)
return parser.items

resulting_versions = []
if parser.items:
for version in parser.items:
max_float = float(version)
resulting_versions.append(str(f'{max_float:2.2f}'))
return resulting_versions


class DebianImageProvider(ImageProviderBase):
Expand Down Expand Up @@ -378,31 +389,22 @@ class OpenSUSEImageProvider(ImageProviderBase):

def __init__(self, version='[0-9]{2}.[0-9]{1}', build=None, arch=DEFAULT_ARCH):
super(OpenSUSEImageProvider, self).__init__(version, build, arch)
self.url_versions = 'https://download.opensuse.org/repositories/Cloud:/Images:/'
self.url_images = self.url_versions + 'Leap_{version}/images/'
self.url_versions = 'https://download.opensuse.org/pub/opensuse/distribution/leap/'
self.url_images = self.url_versions + '{version}/appliances/'

if not build:
self.image_pattern = 'openSUSE-Leap-(?P<version>{version})-OpenStack.(?P<arch>{arch})-((.)*).qcow2$'
self.image_pattern = 'openSUSE-Leap-(?P<version>{version})-JeOS.(?P<arch>{arch})-OpenStack-Cloud.qcow2$'

else:
self.image_pattern = 'openSUSE-Leap-(?P<version>{version})-OpenStack.' \
'(?P<arch>{arch})-(?P<build>{build}).qcow2$'

@property
def version_pattern(self):
return '^Leap_%s' % self._version
self.image_pattern = 'openSUSE-Leap-(?P<version>{version})-JeOS.(?P<arch>{arch})-{version}' \
'-OpenStack-Cloud-Build(?P<build>{build}).qcow2$'

@staticmethod
def _convert_version_numbers(versions):
"""
Return the pure version numbers
The version pattern return Leap_15.0, Leap_42.0, Leap_XY.Z,
but the actual version will numeric versions only
Return float instead of strings
"""
pattern = r'(^Leap_)?([0-9{2}.[0-9]{1})'
replace = r'\2'
return [float(re.sub(pattern, replace, str(v))) for v in versions]
return [float(v) for v in versions]

def get_versions(self):
versions = super(OpenSUSEImageProvider, self).get_versions()
Expand Down
4 changes: 2 additions & 2 deletions docs/source/quickstart/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -243,8 +243,8 @@ Build and Quality Status
:target: https://copr.fedorainfracloud.org/coprs/g/avocado/avocado-latest/package/python-avocado/
:alt: Copr build

.. image:: https://api.cirrus-ci.com/github/avocado-framework/avocado.svg
:target: https://cirrus-ci.com/github/avocado-framework/avocado
.. image:: https://api.cirrus-ci.com/github/avocado-framework/avocado.svg?branch=92lts
:target: https://cirrus-ci.com/github/avocado-framework/avocado/92lts
:alt: Basic checks on Cirrus CI

.. image:: https://img.shields.io/lgtm/alerts/g/avocado-framework/avocado.svg?logo=lgtm&logoWidth=18
Expand Down
2 changes: 1 addition & 1 deletion selftests/pre_release/tests/cirrusci.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
class CirrusCI(unittest.TestCase):

def test(self):
url = 'https://api.cirrus-ci.com/github/avocado-framework/avocado.json'
url = 'https://api.cirrus-ci.com/github/avocado-framework/avocado.json?branch=92lts'
http_response = download.url_open(url)
self.assertEqual(http_response.code, 200)
content = http_response.read()
Expand Down
6 changes: 3 additions & 3 deletions selftests/pre_release/tests/vmimage.py.data/variants.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,17 +76,17 @@ distro: !mux
opensuse:
name: opensuse
!filter-out : /run/architectures/arm
!filter-out : /run/architectures/aarch64
!filter-out : /run/architectures/i386
!filter-out : /run/architectures/ppc
!filter-out : /run/architectures/ppc64
!filter-out : /run/architectures/ppc64le
!filter-out : /run/architectures/s390x
version: !mux
15.1:
version: 15.1
15.2:
!filter-out : /run/architectures/aarch64
version: 15.2
15.3:
version: 15.3

architectures: !mux
arm:
Expand Down

0 comments on commit 640e856

Please sign in to comment.