Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

[1LP][RFR] Correct the steps for rebuilding the repo. #10070

Merged
merged 1 commit into from
Apr 23, 2020
Merged

[1LP][RFR] Correct the steps for rebuilding the repo. #10070

merged 1 commit into from
Apr 23, 2020

Conversation

jarovo
Copy link
Contributor

@jarovo jarovo commented Apr 21, 2020

The test_rh_updates rebuilds some RPMs to make us able to test CFME should be able to update find to some future version.

The repo with the rpms rebuilidng tool had to be rebuilt (yeah... (: ) because the deps on new CFME 5.11 started being not satisfied. I corrected the steps for rebuilding the repo. This will for sure get handy in future. This is not an ideal system, but at the moment I don't know about how to test updates to some future version nicer way without spending money on time travel machine.

The test_rh_updates started working after the repo has been updated.

{{ pytest: -v cfme/tests/configure/test_register_appliance.py::test_rh_updates }}

@dajoRH dajoRH added the lint-ok label Apr 21, 2020
@jarovo jarovo changed the title Correct steps for rebuilding the repo. Correct the steps for rebuilding the repo. Apr 22, 2020
@jarovo jarovo changed the title Correct the steps for rebuilding the repo. [RFR] Correct the steps for rebuilding the repo. Apr 23, 2020
The test_rh_updates rebuilds some RPMs to make us able to test CFME should be able to update find to some future version.

The repo with the rpms rebuilidng tool had to be rebuilt (yeah... (: ) because the deps on new CFME 5.11 started being not satisfied. I corrected the steps for rebuilding the repo. This will for sure get handy in future. This is not an ideal system, but at the moment I don't know about how to test updates to some future version nicer way without spending money on time travel machine.

The test_rh_updates started working after the repo has been updated.
@dajoRH
Copy link
Contributor

dajoRH commented Apr 23, 2020

I detected some fixture changes in commit 8c9649c

The local fixture appliance_preupdate is used in the following files:

  • cfme/tests/configure/test_register_appliance.py
    • test_rh_updates

Please, consider creating a PRT run to make sure your fixture changes do not break existing usage 😃

@john-dupuy john-dupuy added the doc label Apr 23, 2020
@john-dupuy john-dupuy changed the title [RFR] Correct the steps for rebuilding the repo. [1LP][RFR] Correct the steps for rebuilding the repo. Apr 23, 2020
@dajoRH dajoRH removed the doc label Apr 23, 2020
@jawatts jawatts added the manual For things concerning manual test definitions label Apr 23, 2020
@jawatts jawatts merged commit 6fc7ef8 into ManageIQ:master Apr 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lint-ok manual For things concerning manual test definitions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants