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
sysconfig/cloudinit.go: measure (but don't use) gadget cloud-init datasource #10572
sysconfig/cloudinit.go: measure (but don't use) gadget cloud-init datasource #10572
Conversation
2b85e92
to
76823fd
Compare
I opened #10573 which is the full WIP branch that is based on top of this to demonstrate how this code is used, specifically see https://github.com/snapcore/snapd/pull/10573/files#diff-3450862368a77e8ae5d3817feb24a674eadc445a262c1f086d5a3e1a6174ad21L185-R446 for how this is used for decision making |
…asource In a future commit, we will use the result of measuring the datasource settings for the cloud-init config from the gadget, but for now just introduce the machinery to read it. Signed-off-by: Ian Johnson <ian.johnson@canonical.com>
76823fd
to
020a1f9
Compare
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.
lgtm
…_list We previously would just create a list with the duplicated names in it, which didn't break anything but may be confusing for some other bits of code later on. Instead trim out duplicates using a map instead of appending directly into the resultant list. Signed-off-by: Ian Johnson <ian.johnson@canonical.com>
Signed-off-by: Ian Johnson <ian.johnson@canonical.com>
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.
Thank you!
In a future commit, we will use the result of measuring the datasource settings
for the cloud-init config from the gadget, but for now just introduce the
machinery to read it.