Skip to content

Conversation

@fjvela
Copy link
Contributor

@fjvela fjvela commented Sep 8, 2021

Description

Update GCP Storage Bucket binding:

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1125

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

@fjvela fjvela marked this pull request as ready for review September 15, 2021 19:45
@fjvela fjvela requested review from a team as code owners September 15, 2021 19:45
@yaron2
Copy link
Member

yaron2 commented Sep 17, 2021

@beiwei30 can you please review this?

@yaron2
Copy link
Member

yaron2 commented Sep 23, 2021

@fjvela please resolve the conflict and we'll move on to merging. thanks!

@beiwei30 beiwei30 self-requested a review September 27, 2021 09:49
Copy link
Member

@beiwei30 beiwei30 left a comment

Choose a reason for hiding this comment

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

@fjvela Javier, your change looks good to me except for this minor issue.

@codecov
Copy link

codecov bot commented Sep 28, 2021

Codecov Report

Merging #1128 (3fd7dd3) into master (905cbc0) will increase coverage by 0.56%.
The diff coverage is 52.56%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1128      +/-   ##
==========================================
+ Coverage   33.77%   34.34%   +0.56%     
==========================================
  Files         138      140       +2     
  Lines       11686    11993     +307     
==========================================
+ Hits         3947     4119     +172     
- Misses       7329     7450     +121     
- Partials      410      424      +14     
Impacted Files Coverage Δ
bindings/azure/blobstorage/blobstorage.go 14.61% <0.00%> (-0.21%) ⬇️
bindings/azure/eventgrid/eventgrid.go 3.84% <0.00%> (-0.10%) ⬇️
bindings/azure/eventhubs/eventhubs.go 16.66% <0.00%> (-0.66%) ⬇️
...indings/azure/servicebusqueues/servicebusqueues.go 14.66% <0.00%> (-0.41%) ⬇️
bindings/azure/storagequeues/storagequeues.go 37.61% <0.00%> (-1.81%) ⬇️
pubsub/azure/eventhubs/eventhubs.go 23.40% <0.00%> (-0.51%) ⬇️
pubsub/azure/servicebus/servicebus.go 30.19% <0.00%> (-0.30%) ⬇️
state/azure/tablestorage/tablestorage.go 11.86% <0.00%> (-0.21%) ⬇️
state/mongodb/mongodb.go 16.29% <0.00%> (-0.48%) ⬇️
bindings/gcp/bucket/bucket.go 17.39% <19.32%> (+11.67%) ⬆️
... and 7 more

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 e41acc4...3fd7dd3. Read the comment docs.

Copy link
Member

@beiwei30 beiwei30 left a comment

Choose a reason for hiding this comment

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

LGTM now, Thanks Javier for this effort.

@yaron2 yaron2 merged commit 310b4fd into dapr:master Sep 30, 2021
@yaron2
Copy link
Member

yaron2 commented Sep 30, 2021

Merged. Thank you @fjvela for your contribution, and thank you @beiwei30 for the review.

Taction added a commit to Taction/components-contrib that referenced this pull request Oct 6, 2021
* master:
  fix 'durable' setting in RabbitMQ (dapr#1183)
  Support setting up Azure IoT Hub for integration testing with EventHubs components (dapr#1174)
  gcp bucket update  (dapr#1128)
  ok-to-test changes for components-contrib (dapr#1091)
  version stored procedure due to previous changes made (dapr#1176)
  Fix existing integration tests (dapr#1173)
  Add state alicloud.tablestore (dapr#1116)
  Add dapr user agent for Azure Components (dapr#1152)
  Support using setup-azure-conf-test.sh with existing Service Principal (dapr#1158)
  Add Event Hubs conformance tests to conformance.yml workflow (dapr#1164)
  restructure code and add comments per code review
  cover all concurrency + etag cases
  Fix EOF linebreaks
  Adds first-write conformance test
  update metadata key
  fix linter warnings
  TableStorage conformance tests pass
  TableStore config
  tablestorage conformance WIP
  ci: standard linter config (dapr#1102)

# Conflicts:
#	secretstores/hashicorp/vault/vault.go
@fjvela fjvela deleted the binding-gcp-bucket-update branch October 31, 2021 07:06
amimimor pushed a commit to amimimor/components-contrib that referenced this pull request Dec 9, 2021
* gcp bucket update create operation: support upload files in base64

* gcp bucket add get operation

* gcp bucket add delete operation

* gcp bucket add list operation

* gcp bucket fix lint

* gcp bucked add backward compatibility 'key' replace 'name'

* gcp bucket fix lint

* gcp bucket create operation return object url to download

* gcp fix lint

* gcp fix lint

* gcp bucket fix lint

* gcp bucket fix lint

* gcp bucket fix error msg

* rerun checks

Co-authored-by: Yaron Schneider <yaronsc@microsoft.com>
Co-authored-by: Long Dai <long0dai@foxmail.com>
Co-authored-by: Ian Luo <ian.luo@gmail.com>
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.

Update GCP Storage Bucket

4 participants