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

Allow both A and PTR to be updated from SL CCI DNS sync #185

Closed
briancline opened this issue Sep 11, 2013 · 5 comments
Closed

Allow both A and PTR to be updated from SL CCI DNS sync #185

briancline opened this issue Sep 11, 2013 · 5 comments

Comments

@briancline
Copy link
Member

Instead of having to invoke sl cci dns sync -a -y yolo01 twice, once for -a to set the A record, and again with --ptr to set the PTR record, it'd be ideal if these weren't mutually exclusive options as described in the help.

For instance, updating both the A and PTR records thusly:

sl cci dns sync -a --ptr -y yolo01

We'd obviously still want to keep them separate as they are now to maintain backward compatibility, but even if it requires an additional API call—which are fairly quick on these—it seems to be the most intuitive use case.

@sudorandom
Copy link
Contributor

This describes how sl cci dns sync already works

@sudorandom
Copy link
Contributor

Additionally, if you don't provide --a or --ptr it will sync both types of records. I'm going to close this as not an issue.

@briancline
Copy link
Member Author

In that case, we should update the help text to reflect what types of records get updated by default when neither -a nor --ptr are specified.

@sudorandom
Copy link
Contributor

This should be resolved from pull request #188

@briancline
Copy link
Member Author

Thanks brotato.

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

No branches or pull requests

2 participants