Skip to content

Commit

Permalink
Remove CentOS 7 tests
Browse files Browse the repository at this point in the history
CentOS 7 is EOL, and using the vault URLs is somewhat impolite as they
aren't cached.

[noissue]

(cherry picked from commit 99d657d)
  • Loading branch information
dralley authored and patchback[bot] committed Jul 2, 2024
1 parent 4b191c5 commit e4558c5
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 16 deletions.
5 changes: 3 additions & 2 deletions pulp_rpm/tests/functional/api/test_sync.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@

from pulp_rpm.tests.functional.constants import (
AMAZON_MIRROR,
CENTOS7_OPSTOOLS_URL,
PULP_TYPE_ADVISORY,
PULP_TYPE_MODULEMD,
PULP_TYPE_PACKAGE,
Expand Down Expand Up @@ -741,13 +740,15 @@ def test_sha_checksum(init_and_sync):
init_and_sync(url=RPM_SHA_FIXTURE_URL)


@pytest.mark.skip("TODO: Need a new test fixture")
@pytest.mark.parallel
def test_one_nevra_two_locations_and_checksums(init_and_sync):
"""Sync a repository known to have one nevra, in two locations, with different content.
While 'odd', this is a real-world occurrence.
"""
init_and_sync(url=CENTOS7_OPSTOOLS_URL, policy="on_demand")
# init_and_sync(url=CENTOS7_OPSTOOLS_URL, policy="on_demand")
pass


@pytest.mark.parallel
Expand Down
4 changes: 1 addition & 3 deletions pulp_rpm/tests/functional/constants.py
Original file line number Diff line number Diff line change
Expand Up @@ -604,13 +604,11 @@
RPM_ONLY_METADATA_REPO_URL = urljoin(PULP_FIXTURES_BASE_URL, "rpm-unsigned-meta-only")

AMAZON_MIRROR = "http://amazonlinux.us-east-1.amazonaws.com/2/core/latest/x86_64/mirror.list"
CENTOS7_URL = "http://mirror.centos.org/centos-7/7/os/x86_64/"
CENTOS7_OPSTOOLS_URL = "http://mirror.centos.org/centos/7/opstools/x86_64/"
CENTOS8_STREAM_BASEOS_URL = "http://vault.centos.org/centos/8-stream/BaseOS/x86_64/os/"
CENTOS8_STREAM_APPSTREAM_URL = "http://vault.centos.org/centos/8-stream/AppStream/x86_64/os/"
CENTOS9_STREAM_BASEOS_URL = "http://mirror.stream.centos.org/9-stream/BaseOS/x86_64/os/"
CENTOS9_STREAM_APPSTREAM_URL = "http://mirror.stream.centos.org/9-stream/AppStream/x86_64/os/"
EPEL7_URL = "https://dl.fedoraproject.org/pub/epel/7/x86_64/"
EPEL8_URL = "https://dl.fedoraproject.org/pub/epel/8/Everything/x86_64/"
EPEL8_MIRRORLIST_URL = "https://mirrors.fedoraproject.org/mirrorlist?repo=epel-8&arch=x86_64"
RAWHIDE_KICKSTART_URL = (
"https://dl.fedoraproject.org/pub/fedora/linux/development/rawhide/Server/x86_64/os/"
Expand Down
4 changes: 0 additions & 4 deletions pulp_rpm/tests/performance/test_publish.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,8 @@

from pulp_rpm.tests.functional.constants import (
RPM_PACKAGE_CONTENT_NAME,
CENTOS7_URL,
CENTOS8_STREAM_APPSTREAM_URL,
CENTOS8_STREAM_BASEOS_URL,
EPEL7_URL,
)


Expand Down Expand Up @@ -83,8 +81,6 @@ def _extra_test(publication_href):
@pytest.mark.parametrize(
"url,extra_tests",
[
(EPEL7_URL, None),
(CENTOS7_URL, None),
(CENTOS8_STREAM_BASEOS_URL, "centos_8stream_baseos_extra_tests"),
(CENTOS8_STREAM_APPSTREAM_URL, None),
(CENTOS8_STREAM_BASEOS_URL, None),
Expand Down
5 changes: 1 addition & 4 deletions pulp_rpm/tests/performance/test_pulp_to_pulp.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,13 @@
import pytest

from pulp_rpm.tests.functional.constants import (
CENTOS7_URL,
CENTOS8_STREAM_BASEOS_URL,
CENTOS8_STREAM_APPSTREAM_URL,
)


@pytest.mark.parallel
@pytest.mark.parametrize(
"url", [CENTOS7_URL, CENTOS8_STREAM_BASEOS_URL, CENTOS8_STREAM_APPSTREAM_URL]
)
@pytest.mark.parametrize("url", [CENTOS8_STREAM_BASEOS_URL, CENTOS8_STREAM_APPSTREAM_URL])
def test_pulp_to_pulp(
url,
init_and_sync,
Expand Down
4 changes: 1 addition & 3 deletions pulp_rpm/tests/performance/test_sync.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
RHEL8_BASEOS_CDN_URL,
RPM_KICKSTART_CONTENT_NAME,
RPM_KICKSTART_COUNT,
CENTOS7_URL,
CENTOS8_STREAM_APPSTREAM_URL,
CENTOS8_STREAM_BASEOS_URL,
EPEL8_MIRRORLIST_URL,
Expand All @@ -34,9 +33,8 @@ def parse_date_from_string(s, parse_format="%Y-%m-%dT%H:%M:%S.%fZ"):
@pytest.mark.parametrize(
"url,policy,check_dist_tree,resync",
[
(CENTOS7_URL, "on_demand", True, True),
pytest.param(
CENTOS7_URL,
CENTOS8_STREAM_BASEOS_URL,
"immediate",
True,
True,
Expand Down

0 comments on commit e4558c5

Please sign in to comment.