Convert activation key upgrade scenario to new format

[synkd]

Problem Statement

The activation key upgrade scenario is currently failing and uses the old upgrade test format.

Solution

Convert the test to the new format using the shared_resource framework and converting the pre-upgrade test to a pytest fixture.

Remove entitlement-related steps as entitlement mode has been removed from Satellite 6.16+.

Related Issues

This test currently fails on the final org.delete() step due to SAT-28990.

[synkd]

trigger: test-robottelo
pytest: tests/new_upgrades/test_activation_key.py::test_ak_upgrade_scenario

[devendra104]

trigger: test-robottelo
pytest: tests/new_upgrades/test_activation_key.py::test_ak_upgrade_scenario

[synkd]

trigger: test-robottelo
pytest: tests/new_upgrades/test_activation_key.py

[synkd]

trigger: test-robottelo
pytest: tests/new_upgrades/test_activation_key.py
env:
BROKER_SSH_BACKEND: hussh

[Satellite-QE]

PRT Result

Build Number: 9277
Build Status: UNSTABLE
PRT Comment: pytest tests/new_upgrades/test_activation_key.py --external-logging
Test Result : ================= 1 failed, 18 warnings in 2556.00s (0:42:36) ==================

[vsedmik]

Looks good to me. 👍

I have thrown a couple of proposals for improvement but I understand they are out of the scope of this PR and should be rather handled by product teams.

[vsedmik]

This creation should fail since 6.17 (both, CV and LCE, is required). Could we make this creation a bit future-proof by providing it with the Library LCE?

Suggested change

	ak = target_sat.api.ActivationKey(
	content_view=cv, organization=org, name=f'{test_name}_ak'
	).create()
	ak = target_sat.api.ActivationKey(
	content_view=cv, environment=org.library.id, organization=org, name=f'{test_name}_ak'
	).create()

[synkd]

@vsedmik Sure thing, thanks for the feedback. My first thought was that we might want to create a new LCE, but since the post-upgrade section deletes the org at the end, I don't think that's actually necessary.

[vsedmik]

@synkd That was my first thought too, but didn't want to bother with that and CV promotion. Library will do job too with less pain.

[synkd]

trigger: test-robottelo
pytest: tests/new_upgrades/test_activation_key.py
env:
BROKER_SSH_BACKEND: hussh

[Satellite-QE]

PRT Result

Build Number: 9355
Build Status: UNSTABLE
PRT Comment: pytest tests/new_upgrades/test_activation_key.py --external-logging
Test Result : ================= 1 failed, 18 warnings in 2734.19s (0:45:34) ==================