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

chore: Resolve issues and document account resource #3360

Merged
merged 3 commits into from
Jan 30, 2025

Conversation

sfc-gh-jcieslak
Copy link
Collaborator

@sfc-gh-jcieslak sfc-gh-jcieslak commented Jan 27, 2025

Changes

  • Adjust task test as part of SNOW-1843489 (it turned out the logic is correct, but the behavior was confusing; documented in code)
  • Adjusted ticket numbers to corresponding issues for account/task
  • Add a note in the account documentation about undetected changes for certain fields (SNOW-1299979)

For future

  • When account issues will be addressed change ticket numbers.

@sfc-gh-jcieslak sfc-gh-jcieslak marked this pull request as ready for review January 29, 2025 14:20
@sfc-gh-jcieslak sfc-gh-jcieslak force-pushed the report-issues-with-objects branch from a87576e to 430071c Compare January 29, 2025 14:21
Copy link

Integration tests cancelled for 430071cd1afbddee2662438a0bbe306be4e58722

pkg/resources/task_acceptance_test.go Outdated Show resolved Hide resolved
docs/resources/account.md Show resolved Hide resolved
Copy link

Integration tests cancelled for 99016e21c4b25f4566b7b47dd95dc7d8e57f9fa7

@sfc-gh-jcieslak sfc-gh-jcieslak merged commit 46b7a9d into dev Jan 30, 2025
8 of 9 checks passed
@sfc-gh-jcieslak sfc-gh-jcieslak deleted the report-issues-with-objects branch January 30, 2025 12:22
sfc-gh-jcieslak pushed a commit that referenced this pull request Feb 5, 2025
🤖 I have created a release *beep* *boop*
---


##
[1.0.3](v1.0.2...v1.0.3)
(2025-02-05)


### 🔧 **Misc**

* Add basic performance tests
([#3349](#3349))
([c57346b](c57346b))
* Add notes about managing grants on hybrid table
([#3368](#3368))
([e5e98bd](e5e98bd))
* Add preview and stable categories to resources and data sources in
docs
([#3370](#3370))
([b061434](b061434))
* Fix docs and update the protected users list
([#3365](#3365))
([f23e8cb](f23e8cb))
* Grant ownership common use cases
([#3356](#3356))
([97813b6](97813b6))
* Metric level fix with object renaming guide
([#3376](#3376))
([629ff92](629ff92))
* Move technical guides to the guide directory
([#3371](#3371))
([2b95809](2b95809))
* Prepare authentication methods guide
([#3364](#3364))
([c4a1c5f](c4a1c5f))
* Publish Performance tests summary
([#3359](#3359))
([d30d002](d30d002))
* Resolve issues and document account resource
([#3360](#3360))
([46b7a9d](46b7a9d))
* Upgrade tfplugindocs
([#3361](#3361))
([89f2b0a](89f2b0a))


### 🐛 **Bug fixes:**

* change EnforceNetworkRulesForInternalStages ddl to parameter
([#3343](#3343))
([beb01c7](beb01c7))

---
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