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

Add pub_key_ed25519 to cc_phone_home #237

Merged

Conversation

dhensby
Copy link
Contributor

@dhensby dhensby commented Mar 9, 2020

Initial pass at adding the ed25519 host key to allowed phone_home payload keys.

LP: #1866671

@dhensby
Copy link
Contributor Author

dhensby commented Mar 9, 2020

Hi team!

This is my first pass at adding this to phone_home; I'm not sure if there are tests for this feature or what else is required. Happy to improve the PR following any pointers and/or guidance.

Thanks

@OddBloke OddBloke self-assigned this Mar 10, 2020
@OddBloke OddBloke added the CLA signed The submitter of the PR has signed the CLA label Mar 10, 2020
@OddBloke
Copy link
Collaborator

Hi @dhensby, thanks for the contribution! This looks good to me.

There aren't any tests for cc_phone_home currently, but we would love it if you could contribute some. Is that something that you'd be interested in doing? (Either way, given that they don't currently exist, landing this PR is not contingent on you doing so!)

@dhensby
Copy link
Contributor Author

dhensby commented Mar 11, 2020

Hey @OddBloke. I'm afraid my python knowledge is pretty limited so writing a test for this is something that's going to be quite a bit out of my domain of knowledge 😳

Copy link
Collaborator

@OddBloke OddBloke left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Collaborator

@OddBloke OddBloke left a comment

Choose a reason for hiding this comment

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

So this branch is ready to land, but as you can see our "cla-validate" check is failing. This failure is not because of anything you've done, @dhensby, but because we need to fix something up in that check. I'm going to leave this unmerged until #244 lands, so that we have a non-synthetic test case to look at.

(I'm marking this "Request changes" only so we don't accidentally merge this. It is ready to land.)

Copy link
Collaborator

@OddBloke OddBloke left a comment

Choose a reason for hiding this comment

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

OK, the CLA fix has landed and is working, so we can land this once CI has passed. (Thanks for putting up with this noise!)

@dhensby dhensby force-pushed the fix-1866671-ed25519-key-to-phone-home branch from e9839f2 to d27a4ab Compare March 11, 2020 14:45
@OddBloke
Copy link
Collaborator

(For future reference, we squash-merge into master, so maintaining a clean commit history in your PR branch isn't necessary. But I appreciate the effort! 👍)

@OddBloke OddBloke merged commit d5dbbd1 into canonical:master Mar 11, 2020
@dhensby dhensby deleted the fix-1866671-ed25519-key-to-phone-home branch March 11, 2020 15:00
@dhensby
Copy link
Contributor Author

dhensby commented Mar 11, 2020

Thanks for getting this in. Just for my reference what is the kind of timeline we can expect for something like this to start making its way into a release and onto servers?

@OddBloke
Copy link
Collaborator

Thanks for getting this in. Just for my reference what is the kind of timeline we can expect for something like this to start making its way into a release

The next cloud-init release will be 20.2. We don't have a set release date for it, but you can expect it sometime in Q2.

and onto servers?

That rather depends on what distribution you're using. For Ubuntu, we regularly backport upstream snapshots to all stable releases (i.e. 16.04 and later), and I would expect that next to happen soon after focal's release towards the end of April. (As these are snapshots, you wouldn't be waiting for 20.2 to drop.) Once backported, it would make its way into the official Ubuntu cloud images, and then onto servers deployed from them.

(I can't speak with authority for any other distributions, but if you give me a specific one then I can ask around.)

@dhensby
Copy link
Contributor Author

dhensby commented Mar 11, 2020

That's great, about all I need to know. It's not an essential need for us but just good to know. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA signed The submitter of the PR has signed the CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants