feat!: Change default user and hostname to qcom#316
feat!: Change default user and hostname to qcom#316gagath wants to merge 2 commits intoqualcomm-linux:mainfrom
Conversation
gagath
force-pushed
the
change-default-user-hostname
branch
from
4c00dba to
f5bfa80
Compare
|
The qemu test failed because of matching the previous "debian" host instead of "qcom". Amended the commit to fix this. |
|
CI is green, marking as ready for review. |
lool
left a comment
lool
left a comment
There was a problem hiding this comment.
We need to update at least README, perhaps other places
This is a fairly invasive change; I'm not sure how we broadcast this more widely
|
The README was updated for this PR, I added it to the original cherry-pick. I am not sure how to broadcast to our users that this is changing. |
For consistency in docs and CI, use qcom as default user, password, and hostname, no matter the distro. Signed-off-by: Loïc Minier <loic.minier@oss.qualcomm.com> Signed-off-by: Agathe Porte <agathe.porte@oss.qualcomm.com>
gagath
force-pushed
the
change-default-user-hostname
branch
from
f5bfa80 to
cd554d6
Compare
|
I wonder if we could include usage instructions next to the download (probably just pointing back at README) and a warning message that we'd keep for a month or so for changes like this: |
|
We could also try to unify with meta-qcom (root only I think, but perhaps time to address?) |
|
If we take RaspberryPi as a reference there is the EDIT: if you meant to add a qcom user to meta-qcom, sure. I am not sure about shipping sudo though. |
This commit can be reverted when users have integrated the change in their workflows in some time. Signed-off-by: Agathe Porte <agathe.porte@oss.qualcomm.com>
Yeah, I meant having the same non-root user in meta-qcom and qcom-deb-images so that user experience and our CI stay close; meta-qcom might be root-only right now, which probably needs fixing |
|
Added an extra commit to notify users both above the download URL and at the top of the README. The commit can be reverted once users are used to the change. |
Thanks! This might be enough to broadcast Let's just perhaps make sure meta-qcom folks would be happy with username/password "qcom" too, I'd hate changing this back and forth |
|
I will let this PR sleep until we are sure the new user/password make consensus. |
|
Seems we have reached consensus, marking back as ready to review. |
|
@lool I let you the honor of merging this. :) |
|
This is ready for merging, but I've been waiting to have a mechanism to broadcast news to people downloading the builds from CI which IT is planning to implement. |
2 participants
For consistency in docs and CI, use qcom as default user, password, and hostname, no matter the distro.
Cherry-picked from #314, and added missing update to README with the new user/password.