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

Create a Jenkins Job that creates an inventory list #26

Merged
merged 1 commit into from
Apr 24, 2020

Conversation

samuel-rubin
Copy link
Contributor

This is a Jenkins job that gets all the nodes that fit a specified label and do multiple things:

  • It will create an inventory ini file that is readable by ansible tower
  • It will create a markdown table that gives a summary of the machines
  • It will send a slack message listing all of the machines that are offline and their reason

This job will also push the inventory ini and summary table to either push them to git
or it will archive them.

Signed-off-by: Samuel Rubin [email protected]

@samuel-rubin
Copy link
Contributor Author

This is could be related to #25

@samuel-rubin
Copy link
Contributor Author

@AdamBrousseau

@karianna karianna added the enhancement New feature or request label Mar 27, 2019
@karianna
Copy link
Contributor

Paging @sxa555 - This ties into infra

GIT_FOLDER: String - The folder that the files will be saved to. It defaults to a null string
*/

@Library('NodeHelper-Sam') _
Copy link
Contributor

Choose a reason for hiding this comment

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

Might want to remove your name here. :)

SETUP_LABEL: String - the label of the node that this job runs on. The default is worker
LABEL: String - the label of the machines that this job will look for
CREATE_INVENTORY_INI: Boolean - This toggles if the job will create the inventory ini file
CREATE_INVENTORY_SUMMARY: Boolean - This toggles if the summary talbe will be generated
Copy link
Contributor

Choose a reason for hiding this comment

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

spelling of table

CREATE_INVENTORY_INI: Boolean - This toggles if the job will create the inventory ini file
CREATE_INVENTORY_SUMMARY: Boolean - This toggles if the summary talbe will be generated
SLACK_CHANNEL: String - What channel a list of problematic machies gets sent to. If this is empty, no slack message will be sent
SSH_CREDENTIALS: Credentials - The credentials needed to push the files to git
Copy link
Contributor

Choose a reason for hiding this comment

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

spelling or machines

This is a Jenkins job that gets all the nodes that fit a specified label and do multiple things:
-	It will create an inventory ini file that is readable by ansible tower
-	It will create a markdown table that gives a summary of the machines
-	It will send a slack message listing all of the machines that are offline and their reason

This job will also push the inventory ini and summary table to either push them to git
 or it will archive them.

Signed-off-by: Samuel Rubin <[email protected]>
/*
Parameters:
SETUP_LABEL: String - the label of the node that this job runs on. The default is worker
LABEL: String - the label of the machines that this job will look for
Copy link
Contributor

@jdekonin jdekonin Apr 2, 2019

Choose a reason for hiding this comment

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

LABEL can be used as a filter to only get specific machines; like ci.role.perf. Then the output file (groovy code) will only use this subset to operate with. Is this correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Correct

@samuel-rubin
Copy link
Contributor Author

I fixed the spelling errors

@karianna karianna added this to the April 2020 milestone Apr 24, 2020
@karianna karianna requested a review from jdekonin April 24, 2020 10:08
Copy link
Contributor

@jdekonin jdekonin left a comment

Choose a reason for hiding this comment

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

I thought this had been merged actually. Looks good.

@jdekonin jdekonin merged commit 637c9a3 into adoptium:master Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants