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

added functionality for add/delete node(s) #166

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

stephenranjit
Copy link

Hi, This tool is very useful for my team, Thank you!. I've added CLI interface for adding/deleting node(s).

  1. Using a json file for bulk node creation:
    bin/jenkinscli node create_new_node -c <path to yaml file with Jenkins credentials>/login.yml --json <path to json file with a list of tmp machines>host-list.json
  2. Create one node:
    jenkinscli node create_new_node -c login.yml --name tmppn0019648001.lab.in --credentials_id e7df46-278c-459d-8cfd-ec93b21a54 --labels test --executors 2
  3. delete node command also has similar construct

login yaml file example config/login.yml.example
tmp host list example : config/host-list.json

could you please review my code and accept the changes? Thanks again!

"host_list":[
"xxxx"
]
}
Copy link
Owner

Choose a reason for hiding this comment

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

Can we indent this file so it is easier to read?

Copy link
Author

Choose a reason for hiding this comment

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

fixed.

@stephenranjit
Copy link
Author

eb045a5 has all the changes you asked to be made. Please check when you find time. thanks again!

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

Successfully merging this pull request may close these issues.

2 participants