-
Notifications
You must be signed in to change notification settings - Fork 23
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
SUP-4604 - Deprecation of primary_server_port_check task [st1105] #186
SUP-4604 - Deprecation of primary_server_port_check task [st1105] #186
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please run a validate -a when making any changes in ruby after the updates.
Have a look at my comments, but you need to match the array and hash format of the result array, and pay attention to the bracket types and key pointers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made a comment, the solution you have is correct but i have a clarification
6069da3
to
134ba2d
Compare
fab03ec
to
160c936
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the commit message to be in the expected format. It should be something like
(SUP-4604) Deprecate primary port check task
This commit deprecates the task to checks the status of the ports on the
primary Puppet server.
The general format is
(JIRA-KEY) A brief description
A longer description after a newline, if necessary.
We use this for Jira integration and generating the changelog. You can use git commit --amend
since this is only one commit.
This commit deprecates the task to checks the status of the ports on the primary Puppet server and updates the readme.
160c936
to
76d8c6b
Compare
Deprecation Message addition to task ST1105 and Readme Update