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.

Sign up for GitHub

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

Jump to bottom

Torpus cli args #11

Merged
merged 14 commits into from
Oct 6, 2018
Merged

Torpus cli args #11

merged 14 commits into from
Oct 6, 2018

Conversation

Torpus
Copy link
Contributor

@Torpus Torpus commented Oct 5, 2018

No description provided.

@Torpus
added cli.py and updated references within other classees to use it
8a9ea54
@Torpus
added cli.py and updated references within other classees to use it
e826285
@Torpus
moved nonlocal variable declaration to top of function definition
ae2413c
@Torpus
added initial stab at handling arrays of containers to monitor
0cfb5ba
@Torpus
added initial stab at handling arrays of containers to monitor
4b510c1
@Torpus
updated monitor argument to take space delimited list
52577bf
@Torpus
updated args to avoid conflict with argparse defaults
4450237
@Torpus
updated placement of api_client
a5fbcae
@Torpus
updated placement of api_client
a576286
@Torpus
updated placement of api_client
79821ee
@Torpus
updated logic to take action only on containers filtered by name
06b832c
@Torpus
updated logic to take action only on containers filtered by name
5e4a869
@Torpus
resolved issue with non-matching names still getting updated
4bf7d94
@Torpus Torpus closed this Oct 5, 2018
@Torpus Torpus reopened this Oct 5, 2018
Torpus
Torpus commented Oct 5, 2018
View reviewed changes
Copy link
Contributor Author

@Torpus Torpus left a comment

Choose a reason for hiding this comment

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

Left the regex I was working on to validate the url argument but commented out the call that checks it. Need to work out the local “unix://path/to/local” syntax before enabling it

@circa10a circa10a merged commit 8c9f944 into pyouroboros:master Oct 6, 2018
@dirtycajunrice dirtycajunrice added the enhancement New feature or request label Jan 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
3 participants
@Torpus @dirtycajunrice @circa10a