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.

Sign up for GitHub

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

Jump to bottom

Remove pg_trgm #816

Closed
bbernays opened this issue May 31, 2022 · 1 comment · Fixed by #821 or #824
Closed

Remove pg_trgm #816

bbernays opened this issue May 31, 2022 · 1 comment · Fixed by #821 or #824
Assignees
@roneli

Comments

@bbernays
Copy link
Collaborator

Describe the Bug

Users are getting postgres permission errors when not using super_user (details: pq: permission denied to create extension "pg_trgm")

Expected Behavior

Users should not require super_user access to use CloudQuery

CloudQuery Version

24.1 and 24.2
@bbernays bbernays added the bug label May 31, 2022
@roneli roneli self-assigned this Jun 1, 2022
roneli added a commit to roneli/cloudquery that referenced this issue Jun 1, 2022
@roneli
fix: Don't fail on PG_TRGM missing
b7d5304 
Closes cloudquery#816
@roneli roneli mentioned this issue Jun 1, 2022
Merged
5 tasks
@roneli
Copy link
Contributor

roneli commented Jun 1, 2022

Fix in #821 will only create extension & index only if it doesn't fail.

  • if migrations already failed it is required to set them to dirty false so migrations can re-run in cleanstate.

roneli added a commit that referenced this issue Jun 1, 2022
@roneli
fix: Don't fail on PG_TRGM missing (#821)
a0672b5 
Closes #816
@cq-bot cq-bot mentioned this issue Jun 1, 2022
Merged
kodiakhq bot pushed a commit that referenced this issue Jun 2, 2022
@cq-bot
chore(main): Release v0.24.6 (#824)
8de364a 
🤖 I have created a release *beep* *boop*
---


### [0.24.6](v0.24.5...v0.24.6) (2022-06-01)


### Features

* Add goroutine count to sentry ([#825](#825)) ([d4d56c2](d4d56c2))


### Bug Fixes

* Bad condition db version analytics ([#829](#829)) ([6cd8d6d](6cd8d6d))
* Changed level of bad config diagnostics ([#822](#822)) ([67b0bd3](67b0bd3))
* **deps:** Update module github.com/cloudquery/cq-provider-sdk to v0.10.7 ([#826](#826)) ([f075726](f075726))
* Don't fail on PG_TRGM missing ([#821](#821)) ([a0672b5](a0672b5)), closes [#816](#816)
* Incorrect diagnostics summary count ([#823](#823)) ([58760b4](58760b4))

---
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 join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@roneli roneli

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@bbernays @roneli