Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade Capsule features test #15726

Merged

Conversation

lpramuk
Copy link
Contributor

@lpramuk lpramuk commented Jul 22, 2024

Problem Statement

PX recommends to consider adding check for refreshing capsule feature to make sure we have same features enabled before and after upgrade.

Solution

New upgrade-scenarios test

Related Issues

@lpramuk lpramuk self-assigned this Jul 22, 2024
@lpramuk lpramuk added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Jul 22, 2024
@lpramuk lpramuk force-pushed the upgrade_capsule_features branch from 0379f21 to 7b2b213 Compare July 22, 2024 14:06
@lpramuk
Copy link
Contributor Author

lpramuk commented Jul 22, 2024

trigger: test-robottelo
pytest: tests/upgrades/test_capsule.py::TestCapsuleFeatures::test_pre_capsule_features -m pre_upgrade

@lpramuk lpramuk marked this pull request as ready for review July 22, 2024 17:25
@lpramuk lpramuk requested a review from a team as a code owner July 22, 2024 17:25
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7812
Build Status: UNSTABLE
PRT Comment: pytest tests/upgrades/test_capsule.py::TestCapsuleFeatures::test_pre_capsule_features -m pre_upgrade --external-logging
Test Result : =================== 8 warnings, 1 error in 597.93s (0:09:57) ===================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jul 22, 2024
@jameerpathan111
Copy link
Contributor

@lpramuk could you please update the PR description? And add references to original issues if any?

@jameerpathan111 jameerpathan111 self-requested a review July 23, 2024 08:42
Copy link
Contributor

@jameerpathan111 jameerpathan111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, waiting for results.

@lpramuk lpramuk force-pushed the upgrade_capsule_features branch from 7b2b213 to 1b46a80 Compare July 24, 2024 20:07
@lpramuk
Copy link
Contributor Author

lpramuk commented Jul 25, 2024

pre_upgrade

$ pytest -v tests/upgrades/test_capsule.py::TestCapsuleFeatures -m pre_upgrade
=========================================== test session starts ============================================
platform linux -- Python 3.11.9, pytest-8.3.1, pluggy-1.5.0 -- /home/lpramuk/.virtualenvs/r615/bin/python
Mandatory Requirements Mismatch: pytest==8.2.2 broker[docker,podman,hussh]==0.5.1 nailgun @ git+https://github.com/SatelliteQE/nailgun.git@master#egg=nailgun airgun @ git+https://github.com/SatelliteQE/airgun.git@master#egg=airgun
Optional Requirements Mismatch: redis==5.0.7 manage==0.1.15 sphinx-autoapi==3.1.2 pre-commit==3.7.1
To update mismatched requirements, run the pytest command with '--update-required-reqs' OR '--update-all-reqs' option.
cachedir: .pytest_cache
shared_function enabled - OFF - scope:  - storage: file
rootdir: /home/lpramuk/git/SatelliteQE/robottelo
configfile: pyproject.toml
plugins: xdist-3.6.1, reportportal-5.4.1, order-1.2.1, cov-5.0.0, services-2.2.1, fixturecollection-0.1.2, ibutsu-2.2.4, mock-3.14.0
collected 2 items / 1 deselected / 1 selected                                                              

tests/upgrades/test_capsule.py::TestCapsuleFeatures::test_pre_capsule_features PASSED                [100%]

============================================= warnings summary =============================================
../../../.virtualenvs/r615/lib/python3.11/site-packages/novaclient/client.py:31
  /home/lpramuk/.virtualenvs/r615/lib/python3.11/site-packages/novaclient/client.py:31: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
    import pkg_resources

../../../.virtualenvs/r615/lib/python3.11/site-packages/pkg_resources/__init__.py:3144
  /home/lpramuk/.virtualenvs/r615/lib/python3.11/site-packages/pkg_resources/__init__.py:3144: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('sphinxcontrib')`.
  Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
    declare_namespace(pkg)

../../../.virtualenvs/r615/lib/python3.11/site-packages/pkg_resources/__init__.py:3144
  /home/lpramuk/.virtualenvs/r615/lib/python3.11/site-packages/pkg_resources/__init__.py:3144: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('zc')`.
  Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
    declare_namespace(pkg)

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=============================== 1 passed, 1 deselected, 3 warnings in 11.55s ===============================

@lpramuk
Copy link
Contributor Author

lpramuk commented Jul 25, 2024

post_upgrade

$ pytest -v tests/upgrades/test_capsule.py::TestCapsuleFeatures -m post_upgrade
=========================================== test session starts ============================================
platform linux -- Python 3.11.9, pytest-8.3.1, pluggy-1.5.0 -- /home/lpramuk/.virtualenvs/r615/bin/python
Mandatory Requirements Mismatch: pytest==8.2.2 airgun @ git+https://github.com/SatelliteQE/airgun.git@master#egg=airgun broker[docker,podman,hussh]==0.5.1 nailgun @ git+https://github.com/SatelliteQE/nailgun.git@master#egg=nailgun
Optional Requirements Mismatch: redis==5.0.7 manage==0.1.15 pre-commit==3.7.1 sphinx-autoapi==3.1.2
To update mismatched requirements, run the pytest command with '--update-required-reqs' OR '--update-all-reqs' option.
cachedir: .pytest_cache
shared_function enabled - OFF - scope:  - storage: file
rootdir: /home/lpramuk/git/SatelliteQE/robottelo
configfile: pyproject.toml
plugins: xdist-3.6.1, reportportal-5.4.1, order-1.2.1, cov-5.0.0, services-2.2.1, fixturecollection-0.1.2, ibutsu-2.2.4, mock-3.14.0
collected 2 items / 1 deselected / 1 selected                                                              

tests/upgrades/test_capsule.py::TestCapsuleFeatures::test_post_capsule_features PASSED               [100%]

============================================= warnings summary =============================================
../../../.virtualenvs/r615/lib/python3.11/site-packages/novaclient/client.py:31
  /home/lpramuk/.virtualenvs/r615/lib/python3.11/site-packages/novaclient/client.py:31: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
    import pkg_resources

../../../.virtualenvs/r615/lib/python3.11/site-packages/pkg_resources/__init__.py:3144
  /home/lpramuk/.virtualenvs/r615/lib/python3.11/site-packages/pkg_resources/__init__.py:3144: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('sphinxcontrib')`.
  Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
    declare_namespace(pkg)

../../../.virtualenvs/r615/lib/python3.11/site-packages/pkg_resources/__init__.py:3144
  /home/lpramuk/.virtualenvs/r615/lib/python3.11/site-packages/pkg_resources/__init__.py:3144: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('zc')`.
  Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
    declare_namespace(pkg)

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=============================== 1 passed, 1 deselected, 3 warnings in 15.69s ===============================

@jameerpathan111 jameerpathan111 merged commit a7ed77e into SatelliteQE:master Jul 25, 2024
8 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 25, 2024
github-actions bot pushed a commit that referenced this pull request Jul 25, 2024
github-actions bot pushed a commit that referenced this pull request Jul 25, 2024
ogajduse pushed a commit that referenced this pull request Jul 25, 2024
Upgrade Capsule features test (#15726)

(cherry picked from commit a7ed77e)

Co-authored-by: Lukas Pramuk <lpramuk@redhat.com>
ogajduse pushed a commit that referenced this pull request Jul 25, 2024
Upgrade Capsule features test (#15726)

(cherry picked from commit a7ed77e)

Co-authored-by: Lukas Pramuk <lpramuk@redhat.com>
ogajduse pushed a commit that referenced this pull request Jul 25, 2024
Upgrade Capsule features test (#15726)

(cherry picked from commit a7ed77e)

Co-authored-by: Lukas Pramuk <lpramuk@redhat.com>
lpramuk added a commit to lpramuk/robottelo that referenced this pull request Aug 5, 2024
github-actions bot pushed a commit that referenced this pull request Aug 5, 2024
github-actions bot pushed a commit that referenced this pull request Aug 5, 2024
github-actions bot pushed a commit that referenced this pull request Aug 5, 2024
Gauravtalreja1 pushed a commit that referenced this pull request Aug 7, 2024
Fix PR #15726 for missing unpushed changes (#15846)

(cherry picked from commit 898a98b)

Co-authored-by: Lukas Pramuk <lpramuk@redhat.com>
Gauravtalreja1 pushed a commit that referenced this pull request Aug 7, 2024
Fix PR #15726 for missing unpushed changes (#15846)

(cherry picked from commit 898a98b)

Co-authored-by: Lukas Pramuk <lpramuk@redhat.com>
Gauravtalreja1 pushed a commit that referenced this pull request Aug 7, 2024
Fix PR #15726 for missing unpushed changes (#15846)

(cherry picked from commit 898a98b)

Co-authored-by: Lukas Pramuk <lpramuk@redhat.com>
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants