Skip to content
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

feat: upgrade to latest Google Cloud Storage library #40

Merged
merged 1 commit into from
Aug 5, 2022

Conversation

Moumouls
Copy link
Member

No description provided.

@Moumouls Moumouls requested a review from mtrezza July 29, 2022 13:19
@Moumouls Moumouls mentioned this pull request Jul 29, 2022
3 tasks
@codecov
Copy link

codecov bot commented Jul 29, 2022

Codecov Report

Merging #40 (95a1c92) into main (5baee63) will not change coverage.
The diff coverage is 87.50%.

@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage   49.25%   49.25%           
=======================================
  Files           1        1           
  Lines          67       67           
=======================================
  Hits           33       33           
  Misses         34       34           
Impacted Files Coverage Δ
index.js 49.25% <87.50%> (ø)

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 5baee63...95a1c92. Read the comment docs.

Copy link
Member

@mtrezza mtrezza left a comment

Choose a reason for hiding this comment

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

Looks good!

@mtrezza
Copy link
Member

mtrezza commented Jul 29, 2022

@Moumouls There are relatively few changes in this PR, considering that @google-cloud/storage jumps from 1.x to 6.x. And we don't have integration tests in this adapter. Did you verify the adapter works with this change?

@mtrezza mtrezza linked an issue Jul 30, 2022 that may be closed by this pull request
3 tasks
@mtrezza
Copy link
Member

mtrezza commented Jul 30, 2022

@lebenitza could you try out this PR and let us know whether it works?

@lebenitza
Copy link

@Moumouls thank you for the PR!

@mtrezza I will try to find some time to test this. I will test it on GKE, I want to make sure Workload Identity works and see how it behaves with directAccess on and off.

@Moumouls
Copy link
Member Author

Moumouls commented Aug 1, 2022

Hi @lebenitza
I'll also test it the week, I'll try it with direct access false and workload identity from a Cloud Run parse Server container

@Moumouls
Copy link
Member Author

Moumouls commented Aug 4, 2022

@mtrezza @lebenitza I can confirm that everything works correctly. I succeed to upload some files on via Google Cloud Run to GCS.

I can also confirm that workload identity works correctly when directAccess is off. All files are private on GCS.

We are good to go here

@mtrezza mtrezza changed the title feat: upgrade gcs version feat: upgrade to latest Google Cloud Storage library Aug 5, 2022
@mtrezza
Copy link
Member

mtrezza commented Aug 5, 2022

Great, just to confirm, can we merge and release this as a feature version (1.2.0) because despite the GCS upgrade from 1.x to 6.x there is no breaking change for Parse Server?

@Moumouls
Copy link
Member Author

Moumouls commented Aug 5, 2022

No breaking changes @mtrezza google has not changed interfaces/features used by the GCS adapter. 1.2 is okay

@mtrezza mtrezza merged commit 00b89b7 into parse-community:main Aug 5, 2022
parseplatformorg pushed a commit that referenced this pull request Aug 5, 2022
# [1.2.0](1.1.3...1.2.0) (2022-08-05)

### Features

* upgrade to latest Google Cloud Storage library ([#40](#40)) ([00b89b7](00b89b7))
@parseplatformorg
Copy link

🎉 This change has been released in version 1.2.0

1242035 pushed a commit to meo-chuot/parse-server-gcs-adapter that referenced this pull request Jul 5, 2023
1242035 pushed a commit to meo-chuot/parse-server-gcs-adapter that referenced this pull request Jul 5, 2023
1242035 pushed a commit to meo-chuot/parse-server-gcs-adapter that referenced this pull request Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade GCS library
4 participants