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

Refs #23819, #24101 - puppet-3 check as it marked for manual_detection #205

Merged

Conversation

kgaikwad
Copy link
Member

No description provided.

@theforeman-bot
Copy link
Member

Issues: #23819 #24101

@theforeman-bot
Copy link
Member

There were the following issues with the commit message:

  • length of the first commit message line for 7f10eb6 exceeds 65 characters

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

@iNecas
Copy link
Member

iNecas commented Jul 31, 2018

ACK code-wise: @ntkathole mind giving feedback on functionality?

@ntkathole
Copy link
Contributor

Tested on both 6.3 and 6.4. Works fine. ACK

Verify puppet and provide upgrade guide for it: 
\ current puppet version: puppet-3.8.6-4.el7sat.noarch                [FAIL]    
Before continuing with upgrade, please make sure you finish puppet upgrade.
--------------------------------------------------------------------------------
Continue with step [Show knowledge base article for troubleshooting]?, [y(yes), n(no), q(quit)] y
Show knowledge base article for troubleshooting:                                
Go to https://access.redhat.com/documentation/en-us/red_hat_satellite/6.3/html/upgrading_and_updating_red_hat_satellite/upgrading_puppet-1
please follow steps from above article to resolve this issue
press ENTER once done.

@iNecas iNecas merged commit 784ea20 into theforeman:master Jul 31, 2018
@iNecas
Copy link
Member

iNecas commented Jul 31, 2018

Thanks @kgaikwad and @ntkathole

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants