-
Notifications
You must be signed in to change notification settings - Fork 511
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
Generate settings.metrics.send-metrics
based on AWS partition for AWS variants
#2247
Conversation
e8b25fc
to
5cd1c19
Compare
It might be a good idea to split |
5cd1c19
to
50a6201
Compare
@@ -0,0 +1,71 @@ | |||
/// This module contains utilities for populating userdata for the admin-container with user SSH keys from IMDS. | |||
use clap::Parser; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Elsewhere we've tried to use argh
for binary size reasons - can we do that here? structopt
is also OK since it's currently in updog
and metricdog
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGMT once existing comments are addressed.
This commit moves prior shibaken functionality to generate admin container userdata to a subcommand.
Use the new shibaken interface to generate userdata for the admin container.
AWS variants now generate the `settings.metrics.send-metrics` setting based on partition information gathered from shibaken.
50a6201
to
eb2bbc9
Compare
Addresses comment by @bcressey and rebases. Moving from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
Issue number: Related to #1255
Description of changes:
This set of changes modifies
settings.metrics.send-metrics
to use a setting-generator for AWS variants. Specifically, we enable metrics-sending by default on AWS variants if the host is in theaws
oraws-us-gov
partitions, according to the IMDS.This information is retrieved by
shibaken
, which required some changes to the interface, as well as the admin user-data generation which uses that interface.Testing done:
send-metrics
successfully uses shibaken in theaws
partitionsend-metrics
successfully uses shibaken in theaws-us-gov
partitionTerms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.