-
Notifications
You must be signed in to change notification settings - Fork 155
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
Lp 1960162 #1181
Lp 1960162 #1181
Conversation
352c286
to
557bd6e
Compare
I think this looks basically ok. Can we change answers/guided-lvm.yaml to request encryption with a password of "passw0rd" so the grepping scripts/runtests.sh does will guard against regression? |
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.
Looks good to me, thanks!
Just one question related to not accepting None
/ null
anymore in /storage/guided
POST endpoint.
Some API methods have sensitive information in the body of the call. Stop logging the body unconditionally.
Add body logging for POST methods that accept a Payload for data. Use repr=False on UbuntuAdvantageForm to not log the token.
* Update subiquity submodule * subiquity_client: change storage/v2/guided from request params to body (canonical/subiquity#1181) * subiquity_client: add Gap object (canonical/subiquity#1176) * ubuntu_desktop_installer: adapt to Disk.objects/partitions API changes * CI: prepare environment for subiquity (canonical/subiquity#1185) * subiquity_client: add Partition.offset * Update snap config
* Update subiquity submodule * subiquity_client: change storage/v2/guided from request params to body (canonical/subiquity#1181) * subiquity_client: add Gap object (canonical/subiquity#1176) * ubuntu_desktop_installer: adapt to Disk.objects/partitions API changes * CI: prepare environment for subiquity (canonical/subiquity#1185) * subiquity_client: add Partition.offset * Update snap config
No description provided.