-
Notifications
You must be signed in to change notification settings - Fork 83
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
Cherry picks 3.2.1 #469
Cherry picks 3.2.1 #469
Conversation
(cherry picked from commit d17c45e)
There were the following issues with the commit message:
If you don't have a ticket number, please create an issue in Redmine. More guidelines are available in Coding Standards or on the Foreman wiki. This message was auto-generated by Foreman's prprocessor |
You will need #468 for passing tests. |
@akofink even though it fixes tests, this branch is part of 1.13/3.2 where the hammer is older, I suspect if we keep the test fixing commit in here it will break in production |
@ehelms Yes, it must be paired with theforeman/hammer-cli#227. You could modify travis to use the appropriate branch in hammer-cli instead of master, I guess. |
I'd recommend pulling that comment and accepting that tests will not run on this branch. |
6b4db6d
to
dee72c8
Compare
There were the following issues with the commit message:
If you don't have a ticket number, please create an issue in Redmine. More guidelines are available in Coding Standards or on the Foreman wiki. This message was auto-generated by Foreman's prprocessor |
Add organization options to all content view filter rule commands (cherry picked from commit 153cbf2)
dee72c8
to
64b0186
Compare
There were the following issues with the commit message:
If you don't have a ticket number, please create an issue in Redmine. More guidelines are available in Coding Standards or on the Foreman wiki. This message was auto-generated by Foreman's prprocessor |
@ehelms its back to the 2 cherry picks and the version bump |
No description provided.