Skip to content

Use yaml for agent config content #37

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

Merged
merged 4 commits into from
Jul 19, 2017
Merged

Use yaml for agent config content #37

merged 4 commits into from
Jul 19, 2017

Conversation

mstemm
Copy link
Contributor

@mstemm mstemm commented Jul 18, 2017

set_agents_config.py/get_agents_config.py currently read and write the
raw json that's inside the api body. This isn't very user friendly.

Change them to work directly with the agent config-as-yaml content
instead. get_agents_config.py will extract the yaml confg from the json
response and print it. set_agents_config.py will take the agent config
from the file on the command line, verify that it can be parsed as yaml,
wrap it in json, and PUT it.

set_agents_config.py/get_agents_config.py currently read and write the
raw json that's inside the api body. This isn't very user friendly.

Change them to work directly with the agent config-as-yaml content
instead. get_agents_config.py will extract the yaml confg from the json
response and print it. set_agents_config.py will take the agent config
from the file on the command line, verify that it can be parsed as yaml,
wrap it in json, and PUT it.
@mstemm mstemm requested a review from philrz July 18, 2017 15:43
If there is no auto config, the response will have an empty list of
files. Handle that and just print a message that there is no config.
@philrz
Copy link
Contributor

philrz commented Jul 18, 2017

@mstemm and I talked through some changes 1-on-1 and I'm now 👍 on this.

mstemm added 2 commits July 18, 2017 15:00
That way you can tell if there's leading/trailing whitespace, or no
whitespace at all.
Instead of app-staging.sysdigcloud.com.
@mstemm mstemm merged commit 9b835c3 into master Jul 19, 2017
@mstemm mstemm deleted the yaml-for-auto-config branch July 19, 2017 15:13
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