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

Update default user to travis, ssh-key issue fix #1043

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

akhileshbhople
Copy link
Contributor

What is the problem that this PR is trying to fix?

The base images created using old cloud.cfg were not able to get new ssh keys provided during instance boot via openstack, this meant only single ssh key could be used to access the image which was used while the image creation. This PR sets the travis user as default user which allows the VM to get keys provided during instance boot.

What approach did you choose and why?

Modified the cloud.cfg template to set travis user as the default user, this allows new ssh-keys to be added to travis user during instance boot.

How can you make sure the change works as expected?

The changes were tested locally by creating new image and instance.

Would you like any additional feedback?

@akhileshbhople
Copy link
Contributor Author

@joepvd Please review the changes.

@joepvd
Copy link
Contributor

joepvd commented Mar 20, 2019

Thanks! I'd like to test this with our different setups, so this does not break anything in a subtle way. I'll get back to this.

@ramereth
Copy link

I can confirm these changes work on the cluster at the OSUOSL.

@ghatwala
Copy link
Contributor

Hey @joepvd - Did you get a chance to validate this PR on your different setup's ?

@ghatwala
Copy link
Contributor

ghatwala commented Apr 8, 2019

@joepvd - Any updates on above ?

@ramereth
Copy link

@joepvd any updates?

@ghatwala
Copy link
Contributor

Any updates from Travis-CI team on this PR ?

@ghatwala
Copy link
Contributor

hey @joshk - Any review comments on this ?

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

Successfully merging this pull request may close these issues.

4 participants