DOC: Remove update image properties step

[khalford]

As image properties are inherited from the base image, we don't need to update them ourselves. We are assuming the base image will always be correct.

[Dmitry-Popovichev]

When going through these steps, I needed to export the Openstack credentials as well. So perhaps we can add that step in around the Openstack Authentication section?

# Export credentials
export OS_AUTH_URL=https://<openstack_url>:5000/v3
export OS_APPLICATION_CREDENTIAL_ID=<app_cred_id>
export OS_APPLICATION_CREDENTIAL_SECRET=<app_cred_secret>

Otherwise everything else lgtm

[jose-caballero]

Hi @khalford
I understand this change is just to make the documentation to reflect reality, correct?

[khalford]

When going through these steps, I needed to export the Openstack credentials as well. So perhaps we can add that step in around the Openstack Authentication section?

# Export credentials export OS_AUTH_URL=https://<openstack_url>:5000/v3 export OS_APPLICATION_CREDENTIAL_ID=<app_cred_id> export OS_APPLICATION_CREDENTIAL_SECRET=<app_cred_secret>

Otherwise everything else lgtm

@Dmitry-Popovichev You don't need to do that if you put your app cred in ~/.config/openstack/clouds.yaml clouds.yaml is the prefered way as environment variables can leak with bash scripts.

[khalford]

Hi @khalford I understand this change is just to make the documentation to reflect reality, correct?

@jose-caballero Yes, I have removed this step because it is not needed. Detail is in the commit message and developer notes

[Dmitry-Popovichev]

When going through these steps, I needed to export the Openstack credentials as well. So perhaps we can add that step in around the Openstack Authentication section?
# Export credentials export OS_AUTH_URL=https://<openstack_url>:5000/v3 export OS_APPLICATION_CREDENTIAL_ID=<app_cred_id> export OS_APPLICATION_CREDENTIAL_SECRET=<app_cred_secret>
Otherwise everything else lgtm

@Dmitry-Popovichev You don't need to do that if you put your app cred in ~/.config/openstack/clouds.yaml clouds.yaml is the prefered way as environment variables can leak with bash scripts.

I did think that and I'm fairly certain I had a clouds.yaml file in the correct path but when creating the image it came up with an error that I forgot to save. I'll approve this PR anyway and we can change the docs later if needs be when we create and test the remaining capi versions.