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

[6.2] Add new hammer options #4655

Merged
merged 1 commit into from
May 9, 2017
Merged

[6.2] Add new hammer options #4655

merged 1 commit into from
May 9, 2017

Conversation

lpramuk
Copy link
Contributor

@lpramuk lpramuk commented May 9, 2017

hammer content-view publish
Added options:
* force-yum-metadata-regeneration

hammer content-view version promote
Added options:
* force-yum-metadata-regeneration
* description
* environment-ids

hammer repository synchronize
Added options:
* validate-contents
* skip-metadata-check

(Fixes #4614)

@lpramuk
Add new hammer options
b733ac5 
hammer content-view publish
  Added options:
    * force-yum-metadata-regeneration

hammer content-view version promote
  Added options:
    * force-yum-metadata-regeneration
    * description
    * environment-ids

hammer repository synchronize
  Added options:
    * validate-contents
    * skip-metadata-check

(Fixes SatelliteQE#4614)
@lpramuk lpramuk added the review label May 9, 2017
@lpramuk lpramuk self-assigned this May 9, 2017
@lpramuk
Copy link
Contributor Author

lpramuk commented May 9, 2017 •
edited

I'm not forgetting about an equivalent to master. Gonna check Satellite Nightly

edit: Satellite Nightly has it. Submitted PR #4656 for master aswell.

@lpramuk lpramuk changed the title Add new hammer options [6.2] Add new hammer options May 9, 2017
@lpramuk lpramuk added 6.2 TestFailure Issues and PRs related to a test failing in automation labels May 9, 2017
@lpramuk lpramuk mentioned this pull request May 9, 2017
Merged
pondrejk
pondrejk approved these changes May 9, 2017
View reviewed changes
Copy link
Contributor

@pondrejk pondrejk left a comment

Choose a reason for hiding this comment

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

ACK

renzon
renzon approved these changes May 9, 2017
View reviewed changes
Copy link
Contributor

@renzon renzon left a comment

Choose a reason for hiding this comment

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

ACK

@renzon renzon merged commit 54bf64c into SatelliteQE:6.2.z May 9, 2017
@renzon renzon removed review TestFailure Issues and PRs related to a test failing in automation labels May 9, 2017
@lpramuk
Copy link
Contributor Author

lpramuk commented May 10, 2017 

$ pytest tests/foreman/cli/test_hammer.py::HammerCommandsTestCase::test_positive_all_options
================================================= test session starts ==================================================
platform linux2 -- Python 2.7.13, pytest-3.0.7, py-1.4.32, pluggy-0.4.0
shared_function enabled - OFF - scope:  - storage: file
rootdir: /home/lpramuk/git/SatelliteQE/robottelo, inifile:
plugins: services-1.1.14, cov-2.3.1
collected 2 items 
2017-05-09 16:57:59 - conftest - DEBUG - Found WONTFIX in decorated tests ['1110476', '1156555', '1269196', '1402826', '1245334', '1221971', '1217635', '1226425', '1199150', '1204686', '1267224', '1103157', '1230902', '1214312', '1079482']

2017-05-09 16:57:59 - conftest - DEBUG - Collected 1 test cases


tests/foreman/cli/test_hammer.py .

@lpramuk lpramuk deleted the 62_hammer_all_options branch June 10, 2020 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@renzon renzon renzon approved these changes

@pondrejk pondrejk pondrejk approved these changes

@elyezer elyezer Awaiting requested review from elyezer

@rochacbruno rochacbruno Awaiting requested review from rochacbruno

Assignees

@lpramuk lpramuk

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@lpramuk @renzon @pondrejk