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

Add support for RHCloud Cloud Connector #920

Merged
merged 10 commits into from
Jun 21, 2023

Conversation

mstrlc
Copy link
Contributor

@mstrlc mstrlc commented May 2, 2023

Description of changes

This PR adds Nailgun support for RHCloud Cloud Connector API.

Functional demonstration

This feature is tested with Robottelo test tests/foreman/api/test_rhc.py::test_positive_configure_cloud_connector,
see SatelliteQE/robottelo#11395 PRT run.

# Running in Robottelo tests/foreman/api/test_rhc.py::test_positive_configure_cloud_connector.
result = module_target_sat.api.RHCloud().enable_connector(organization_id=module_rhc_org.id)
print(result)
{'id': 4, 'targeting_id': 4, 'job_category': 'Maintenance Operations', 'task_id': '97612455-60d8-45e4-904f-d7b067b9e74f', 'task_group_id': 16, 'triggering_id': 4, 'description': 'Configure Cloud Connector', 'concurrency_level': None, 'time_span': None, 'execution_timeout_interval': None, 'password': None, 'key_passphrase': None, 'remote_execution_feature_id': 17, 'effective_user_password': None, 'ssh_user': None, 'time_to_pickup': None}
Additional Information

Nailgun tests addded and passing.

@codecov

This comment was marked as outdated.

@mstrlc mstrlc added CherryPick PR needs CherryPick to previous branches 6.12.z 6.13.z AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing labels May 4, 2023
@mstrlc mstrlc marked this pull request as ready for review May 4, 2023 12:10
nailgun/entities.py Outdated Show resolved Hide resolved
Co-authored-by: Ondřej Gajdušek <[email protected]>
@jyejare jyejare added the 6.14.z label Jun 1, 2023
@mstrlc mstrlc requested a review from ogajduse June 6, 2023 12:29
jameerpathan111
jameerpathan111 previously approved these changes Jun 7, 2023
vsedmik
vsedmik previously approved these changes Jun 16, 2023
nailgun/entities.py Outdated Show resolved Hide resolved
nailgun/entities.py Show resolved Hide resolved
@mstrlc mstrlc dismissed stale reviews from vsedmik and jameerpathan111 via 92586e0 June 20, 2023 13:12
@jameerpathan111 jameerpathan111 merged commit 13c0375 into SatelliteQE:master Jun 21, 2023
github-actions bot pushed a commit that referenced this pull request Jun 21, 2023
* add support for RHCloud Cloud Connector

* remove location_id and organization_id from args

* add Nailgun tests

* add Nailgun tests

* update entity fields

Co-authored-by: Ondřej Gajdušek <[email protected]>

* change the way data are passed to the function

* change the way data are passed to the function

* update test to match change

---------

Co-authored-by: Ondřej Gajdušek <[email protected]>
(cherry picked from commit 13c0375)
github-actions bot pushed a commit that referenced this pull request Jun 21, 2023
* add support for RHCloud Cloud Connector

* remove location_id and organization_id from args

* add Nailgun tests

* add Nailgun tests

* update entity fields

Co-authored-by: Ondřej Gajdušek <[email protected]>

* change the way data are passed to the function

* change the way data are passed to the function

* update test to match change

---------

Co-authored-by: Ondřej Gajdušek <[email protected]>
(cherry picked from commit 13c0375)
mstrlc added a commit to mstrlc/nailgun that referenced this pull request Jun 21, 2023
* add support for RHCloud Cloud Connector

* remove location_id and organization_id from args

* add Nailgun tests

* add Nailgun tests

* update entity fields

Co-authored-by: Ondřej Gajdušek <[email protected]>

* change the way data are passed to the function

* change the way data are passed to the function

* update test to match change

---------

Co-authored-by: Ondřej Gajdušek <[email protected]>
(cherry picked from commit 13c0375)
ogajduse pushed a commit that referenced this pull request Jun 21, 2023
* add support for RHCloud Cloud Connector

* remove location_id and organization_id from args

* add Nailgun tests

* add Nailgun tests

* update entity fields

Co-authored-by: Ondřej Gajdušek <[email protected]>

* change the way data are passed to the function

* change the way data are passed to the function

* update test to match change

---------

Co-authored-by: Ondřej Gajdušek <[email protected]>
(cherry picked from commit 13c0375)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.12.z 6.13.z 6.14.z AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants