Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

fix: Bucket missing region #978

Merged
merged 1 commit into from
Jun 1, 2022

Conversation

roneli
Copy link
Contributor

@roneli roneli commented Jun 1, 2022

🎉 Thank you for making CloudQuery awesome by submitting a PR 🎉

Summary

Fix edge case where bucket region is not set, causing a cascading fail on all relation fetches

Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests. Learn more about testing here 🧪
  • Update the docs by running go run ./docs/docs.go and committing the changes 📃
  • If adding a new resource, add relevant Terraform files in a separate PR 📂
  • Ensure the status checks below are successful ✅

@roneli roneli requested a review from a team as a code owner June 1, 2022 07:59
@roneli roneli requested review from shimonp21 and removed request for a team June 1, 2022 07:59
@yevgenypats yevgenypats merged commit 1467b6e into cloudquery:main Jun 1, 2022
@roneli roneli deleted the fix/bucket_missing_region branch June 1, 2022 08:34
kodiakhq bot pushed a commit that referenced this pull request Jun 1, 2022
🤖 I have created a release *beep* *boop*
---


### [0.12.3](v0.12.2...v0.12.3) (2022-06-01)


### Bug Fixes

* Bucket missing region ([#978](#978)) ([1467b6e](1467b6e))
* Handle panic on Waf logging configuration not found ([#970](#970)) ([8ffb3e6](8ffb3e6))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
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.

2 participants