Skip to content
This repository has been archived by the owner on Jan 12, 2024. It is now read-only.

Switch to using requester pays on storage buckets #23

Merged
merged 22 commits into from
May 24, 2022
Merged

Switch to using requester pays on storage buckets #23

merged 22 commits into from
May 24, 2022

Conversation

zaneselvans
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented May 17, 2022

Codecov Report

Merging #23 (77f9db5) into main (84eaaa8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #23   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files           2        2           
  Lines          42       42           
=======================================
  Hits           42       42           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84eaaa8...77f9db5. Read the comment docs.

@zaneselvans zaneselvans linked an issue May 17, 2022 that may be closed by this pull request
5 tasks
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Usage
-----

Public data and "requester pays"
Copy link
Member

Choose a reason for hiding this comment

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

can we add this stuff into the data access page instead of the readme??

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll break it out into a separate page and link to it. I agree this is too much detail for the README.

Copy link
Member

@cmgosnell cmgosnell left a comment

Choose a reason for hiding this comment

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

some questions....

README.rst Show resolved Hide resolved
.. code:: bash

gcloud auth login
gcloud init
Copy link
Member

Choose a reason for hiding this comment

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

this step asked me some questions:

Pick configuration to use:
 [1] Re-initialize this configuration [default] with new settings 
 [2] Create a new configuration

Copy link
Member

Choose a reason for hiding this comment

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

it also asked:

Pick cloud project to use: 
 [1] northern-hope-350719
 [2] pudl-test
 [3] Enter a project ID
 [4] Create a new project

(I had just made the pudl-test project when i first logged in)

Copy link
Member

Choose a reason for hiding this comment

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

i choose 2

Copy link
Member Author

Choose a reason for hiding this comment

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

Which one did you choose? I think it should be the project you just created, and within which you should have given yourself the permission to spend money.

See the notebook included in this repository for more details.
.. code:: bash

gsutil ls gs://intake.catalyst.coop
Copy link
Member

Choose a reason for hiding this comment

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

okay i got:

(pudl-dev) %0 ~/code% gsutil ls gs://intake.catalyst.coop                                                            christinagosnell@New-Shiny-Thing
BadRequestException: 400 Bucket is a requester pays bucket but no user project provided.
[1]    18496 exit 1     gsutil ls gs://intake.catalyst.coop
``

Copy link
Member Author

Choose a reason for hiding this comment

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

What do you get when you do

gcloud config configurations list

Copy link
Member Author

Choose a reason for hiding this comment

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

(but good to know that the gsutil ls is a sufficient check!)

Copy link
Member

Choose a reason for hiding this comment

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

(pudl-dev) %0 ~/code% gcloud config configurations list                                                              christinagosnell@New-Shiny-Thing
NAME     IS_ACTIVE  ACCOUNT               PROJECT    COMPUTE_DEFAULT_ZONE  COMPUTE_DEFAULT_REGION
default  True       [email protected]  pudl-test

Copy link
Member

@cmgosnell cmgosnell left a comment

Choose a reason for hiding this comment

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

for me, the most confusing part of this was the initial setup of the account and project. It felt very fuck-around-and-find-out.... some lil screenshots would be helpful.

(a part of my problem was i was trying to do this on my non-primary google account so I could read the instructions on all of the links but not actually do anything there.)

and i commented on the several command line hurdles

@zaneselvans zaneselvans merged commit 43a0867 into main May 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set up requester pays
2 participants