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.

Sign up for GitHub

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

Jump to bottom

[cherry-pick] add token for github api calls #9340

Merged
merged 1 commit into from
Feb 1, 2022
Merged

[cherry-pick] add token for github api calls #9340

merged 1 commit into from
Feb 1, 2022

Conversation

peterdragun
Copy link
Contributor

Backporting PR #9326 to 6.10z branch

Test results:

pytest -v tests/foreman/api/test_templatesync.py 
============================================================================= test session starts =============================================================================
platform linux -- Python 3.9.9, pytest-6.2.5, py-1.10.0, pluggy-0.13.1 -- /home/pdragun/.virtualenvs/robottelo/bin/python
cachedir: .pytest_cache
shared_function enabled - OFF - scope:  - storage: file
rootdir: /home/pdragun/Documents/robottelo, configfile: pyproject.toml
plugins: cov-2.12.1, forked-1.3.0, services-2.2.1, reportportal-5.0.8, mock-3.6.1, xdist-2.5.0, ibutsu-2.0.2
collected 29 items                                                                                                                                                            

tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_filtered_templates_from_git PASSED                                               [  3%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_import_filtered_templates_from_git_with_negate PASSED                                            [  6%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_and_associate PASSED                                                             [ 10%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_from_subdirectory PASSED                                                         [ 13%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_filtered_templates_to_localdir PASSED                                            [ 17%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_filtered_templates_negate PASSED                                                 [ 20%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_and_import_with_metadata PASSED                                                  [ 24%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_verbose[True] PASSED                                                 [ 27%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_verbose[False] PASSED                                                [ 31%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_changed_key_true PASSED                                              [ 34%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_changed_key_false PASSED                                             [ 37%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_name_key PASSED                                                      [ 41%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_imported_key PASSED                                                  [ 44%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_file_key PASSED                                                      [ 48%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_corrupted_metadata PASSED                                            [ 51%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_filtered_skip_message PASSED                                         [ 55%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_no_name_error PASSED                                                 [ 58%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_no_model_error PASSED                                                [ 62%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_json_output_blank_model_error PASSED                                             [ 65%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_json_output PASSED                                                               [ 68%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_log_to_production PASSED                                                         [ 72%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_log_to_production PASSED                                                         [ 75%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_all_templates_to_repo[non_empty_repo-http] PASSED                                [ 79%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_all_templates_to_repo[non_empty_repo-ssh] PASSED                                 [ 82%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_all_templates_to_repo[empty_repo-http] PASSED                                    [ 86%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_all_templates_to_repo[empty_repo-ssh] PASSED                                     [ 89%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_all_templates_from_repo PASSED                                                   [ 93%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_negative_import_locked_template PASSED                                                           [ 96%]
tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_locked_template PASSED                                                           [100%]

================================================================= 29 passed, 58 warnings in 187.10s (0:03:07) =================================================================

@peterdragun peterdragun added 6.10.z Introduced in or relating directly to Satellite 6.10 Easy Fix :) Easiest Fix to review and quick merge request. labels Jan 31, 2022
@peterdragun peterdragun requested a review from ogajduse January 31, 2022 14:13
@omkarkhatavkar
Copy link

Can one of the admins verify this patch?

@ogajduse
Copy link
Member

ogajduse commented Feb 1, 2022

@JacobCallahan GH Actions is failing also for the 6.10.z branch. Do we want to backport #9341 also to this branch?

ogajduse
ogajduse approved these changes Feb 1, 2022
View reviewed changes
Copy link
Member

@ogajduse ogajduse left a comment •
edited
Loading

Choose a reason for hiding this comment

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

ACK, CI is failing. In order to fix it, we need to backport #9341 or we can wait for the new pip release.

@mshriver mshriver merged commit 6872f04 into SatelliteQE:6.10.z Feb 1, 2022
@JacobCallahan
Copy link
Member

@JacobCallahan GH Actions is failing also for the 6.10.z branch. Do we want to backport #9341 also to this branch?

Please do and tag me in the pr so I can merge it, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ogajduse ogajduse ogajduse approved these changes

@mshriver mshriver mshriver approved these changes

Assignees
No one assigned
Labels
6.10.z Introduced in or relating directly to Satellite 6.10 Easy Fix :) Easiest Fix to review and quick merge request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@peterdragun @omkarkhatavkar @ogajduse @JacobCallahan @mshriver