Skip to content

Conversation

abikouo
Copy link

@abikouo abikouo commented Aug 17, 2023

Due to limitation with google.cloud collection, this role/playbook cannot delete source storage bucket as we can't list objects from one bucket (cannot delete non empty storage bucket), users also needs to provided list of objects to move.

#5 should be merged first for the CI to pass on this one

Copy link
Member

@gravesm gravesm left a comment

Choose a reason for hiding this comment

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

Can you split the credentials role into a separate PR? I'm not sure if we want to merge the bucket role without the ability to list all objects in a bucket, but that shouldn't hold up the credentials work.

@abikouo
Copy link
Author

abikouo commented Aug 22, 2023

Can you split the credentials role into a separate PR? I'm not sure if we want to merge the bucket role without the ability to list all objects in a bucket, but that shouldn't hold up the credentials work.

#5 has been created for gcp_setup_credentials role

Comment on lines +5 to +7
auth_kind: "{{ gcp_auth_kind }}"
service_account_file: "{{ gcp_cred_file }}"
project: "{{ gcp_project }}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why don't you use the role for creds setting here?

Copy link
Author

Choose a reason for hiding this comment

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

I can also do that, I was just using it for collection role, but it should work too

@gravesm
Copy link
Member

gravesm commented Sep 6, 2023

We can leave this PR open in case things change, but we do not want to merge this as the limitations of the google.cloud collection prevent us from properly implementing this use case.

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.

3 participants