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: Database SDK upgrade #2814

Merged
merged 7 commits into from
May 22, 2024
Merged

feat: Database SDK upgrade #2814

merged 7 commits into from
May 22, 2024

Conversation

sfc-gh-jcieslak
Copy link
Collaborator

@sfc-gh-jcieslak sfc-gh-jcieslak commented May 16, 2024

Summary of changes

  • Added catalog and external volume test clients
  • Added missing fields in the database operations + validations
  • Moved LogLevel and TraceLevel to common types

Test Plan

  • Added / modified unit tests
  • Added / modified integration tests

References

@sfc-gh-jcieslak sfc-gh-jcieslak changed the title Database v1 readiness feat: Database SDK upgrade May 16, 2024
Copy link

Integration tests failure for 75856a6b00d8a2c24f6daab506a847fddf32dc34

@sfc-gh-jcieslak sfc-gh-jcieslak force-pushed the database-v1-readiness branch 2 times, most recently from 2d12dc4 to 19c51e2 Compare May 16, 2024 10:34
Copy link

Integration tests failure for 2d12dc40b191c025900c3157abcc6cdc204a8e29

Copy link

Integration tests failure for 19c51e22448cbd77bb0085ff084df1adcedf0709

Copy link

Integration tests failure for 9fe36274bffd23758c037fa1f578cfac7e243f1d

Copy link

Integration tests success for 464915c8f408b74cb15fb4182a7c2deece4c37f5

pkg/sdk/databases.go Outdated Show resolved Hide resolved
pkg/acceptance/helpers/catalog_client.go Show resolved Hide resolved
pkg/sdk/testint/databases_integration_test.go Outdated Show resolved Hide resolved
pkg/sdk/testint/databases_integration_test.go Outdated Show resolved Hide resolved
pkg/sdk/testint/databases_integration_test.go Show resolved Hide resolved
pkg/sdk/databases.go Show resolved Hide resolved
pkg/sdk/databases.go Show resolved Hide resolved
pkg/sdk/databases.go Show resolved Hide resolved
pkg/sdk/databases.go Show resolved Hide resolved
pkg/sdk/databases.go Show resolved Hide resolved
pkg/sdk/databases.go Show resolved Hide resolved
pkg/sdk/databases_test.go Show resolved Hide resolved
pkg/sdk/testint/databases_integration_test.go Outdated Show resolved Hide resolved
pkg/sdk/testint/databases_integration_test.go Outdated Show resolved Hide resolved
pkg/sdk/testint/databases_integration_test.go Outdated Show resolved Hide resolved
# Conflicts:
#	pkg/acceptance/helpers/test_client.go
#	pkg/resources/database.go
Copy link

Integration tests success for 8a6b67acafafba9dcf5a2dd671ac571ab3f6ce36

@sfc-gh-asawicki sfc-gh-asawicki self-requested a review May 22, 2024 07:44
Copy link

Integration tests failure for 09a52d0c27f184d9c2b3a542030e5a9ea1968e09

@sfc-gh-jcieslak sfc-gh-jcieslak merged commit 750fe37 into main May 22, 2024
8 of 9 checks passed
@sfc-gh-jcieslak sfc-gh-jcieslak deleted the database-v1-readiness branch May 22, 2024 09:01
sfc-gh-jcieslak pushed a commit that referenced this pull request Jun 6, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.92.0](v0.91.0...v0.92.0)
(2024-06-06)


### 🎉 **What's new:**

* Add Api Authentication security integration to sdk
([#2840](#2840))
([57a07ee](57a07ee))
* Add External Oauth security integration to sdk
([#2835](#2835))
([82d1c09](82d1c09))
* add network rules
([#2746](#2746))
([c79fa29](c79fa29))
* Add SCIM and SAML2 security integrations to sdk
([#2799](#2799))
([1312ff1](1312ff1))
* Add Snowflake Oauth security integration to sdk
([#2830](#2830))
([b576f29](b576f29))
* Database resource v1 readiness
([#2834](#2834))
([30fe136](30fe136))
* Database SDK upgrade
([#2814](#2814))
([750fe37](750fe37))


### 🔧 **Misc**

* accept non-pointer values in the generated builder methods
([#2816](#2816))
([c29fbf1](c29fbf1))
* Add a script for creating labels
([#2778](#2778))
([ce0fbad](ce0fbad))
* Adjust before 0.92.0
([#2857](#2857))
([0598656](0598656))
* Continue random ids rework
([#2819](#2819))
([f20940c](f20940c))
* Random ids rework part3
([#2833](#2833))
([36ead85](36ead85))
* Random ids rework part4
([#2837](#2837))
([64518a3](64518a3))
* Update issue for table and warehouse redesign state
([#2845](#2845))
([149e55e](149e55e))


### 🐛 **Bug fixes:**

* Fix failing integration tests
([#2832](#2832))
([2e2ca6c](2e2ca6c))
* Fix QUOTED_IDENTIFIERS_IGNORE_CASE parameter test
([#2841](#2841))
([92ad1d3](92ad1d3))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-release-please[bot]@users.noreply.github.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.

3 participants