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

Transaction in use error when creating role #2197

Closed
3 tasks done
Flo4604 opened this issue Oct 5, 2024 · 21 comments · Fixed by #2278
Closed
3 tasks done

Transaction in use error when creating role #2197

Flo4604 opened this issue Oct 5, 2024 · 21 comments · Fixed by #2278

Comments

@Flo4604
Copy link
Contributor

Flo4604 commented Oct 5, 2024

Preliminary Checks

Reproduction / Replay Link (Optional)

No response

Issue Summary

I randomly got this error when creating a role

{"error":{"code":"INTERNAL_SERVER_ERROR","docs":"https://unkey.dev/docs/api-reference/errors/code/INTERNAL_SERVER_ERROR","message":"target: unkey.-.primary: vttablet: rpc error: code = Aborted desc = transaction 1727848515790095014: in use: for query (CallerID: kzr01kdt6cz27x485f8p)","requestId":"req_2xEK1qBBAYjecN8sMJr7fHbKuajZ"}

Steps to Reproduce

  1. Try to create a role via API Example code of what I did when creating the role:
roles := []string{"admin", "account"}
for _, role := range roles {
    _, err := s.unkey.Permissions.CreateRole(ctx, operations.CreateRoleRequestBody{
    	Name: role,
    })
    
    if err != nil {
    	log.Printf("Error creating role %s", err)
    }
}

Expected behavior

To not get an error

Other information

No response

Screenshots

No response

Version info

-
@Flo4604 Flo4604 added Bug Something isn't working Needs Approval Needs approval from Unkey labels Oct 5, 2024
Copy link

linear bot commented Oct 5, 2024

@chronark
Copy link
Collaborator

chronark commented Oct 5, 2024

This is flaky, right?
Not consistently reproducible?

@chronark
Copy link
Collaborator

chronark commented Oct 5, 2024

/award 150 points

Copy link

oss-gg bot commented Oct 5, 2024

Awarding Flo4604: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/Flo4604

@RajuGangitla
Copy link
Contributor

/assign

Copy link

oss-gg bot commented Oct 5, 2024

Assigned to @RajuGangitla! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

@RajuGangitla RajuGangitla removed their assignment Oct 5, 2024
@Flo4604
Copy link
Contributor Author

Flo4604 commented Oct 5, 2024

This is flaky, right? Not consistently reproducible?

yup, its randomly happening, ill try to find something how its 100% repro

@chronark
Copy link
Collaborator

chronark commented Oct 5, 2024

That'd be great, we can award more points if you can reliably reproduce it, or at least get us closer to the root cause

@KekmaTime
Copy link

/assign

Copy link

oss-gg bot commented Oct 6, 2024

Assigned to @KekmaTime! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

Copy link

oss-gg bot commented Oct 6, 2024

@RajuGangitla, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

@Flo4604
Copy link
Contributor Author

Flo4604 commented Oct 7, 2024

/assign

Copy link

oss-gg bot commented Oct 7, 2024

This issue is already assigned to another person. Please find more issues here.

@Flo4604
Copy link
Contributor Author

Flo4604 commented Oct 7, 2024

/assign

@oss-gg oss-gg bot assigned Flo4604 Oct 7, 2024
Copy link

oss-gg bot commented Oct 7, 2024

Assigned to @Flo4604! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

Copy link

oss-gg bot commented Oct 8, 2024

@KekmaTime, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

Flo4604 added a commit to Flo4604/unkey that referenced this issue Oct 8, 2024
- Fixes unkeyed#2197
- The error is caused by the cache revalidation happening in the background, so we have a racecondition with the other queries we are running in the insertGenericAuditLogs function
Copy link

oss-gg bot commented Oct 8, 2024

@RajuGangitla, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

Copy link

oss-gg bot commented Oct 10, 2024

@KekmaTime, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

@HimanshuGarg5480
Copy link

/assign

Copy link

oss-gg bot commented Oct 10, 2024

This issue is already assigned to another person. Please find more issues here.

@Flo4604
Copy link
Contributor Author

Flo4604 commented Oct 10, 2024

/assign

there is already a pr open for this

github-merge-queue bot pushed a commit that referenced this issue Oct 11, 2024
* fix: transaction ...: in use: for query
- Fixes #2197
- The error is caused by the cache revalidation happening in the background, so we have a racecondition with the other queries we are running in the insertGenericAuditLogs function

* docs: explain for future self

* fix: another racecondition
also bulk insert auditLogs and auditLogTargets

* [autofix.ci] apply automated fixes

---------

Co-authored-by: chronark <dev@chronark.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
AkshayBandi027 pushed a commit to AkshayBandi027/unkey that referenced this issue Oct 14, 2024
* fix: transaction ...: in use: for query
- Fixes unkeyed#2197
- The error is caused by the cache revalidation happening in the background, so we have a racecondition with the other queries we are running in the insertGenericAuditLogs function

* docs: explain for future self

* fix: another racecondition
also bulk insert auditLogs and auditLogTargets

* [autofix.ci] apply automated fixes

---------

Co-authored-by: chronark <dev@chronark.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
AkshayBandi027 pushed a commit to AkshayBandi027/unkey that referenced this issue Oct 14, 2024
* fix: transaction ...: in use: for query
- Fixes unkeyed#2197
- The error is caused by the cache revalidation happening in the background, so we have a racecondition with the other queries we are running in the insertGenericAuditLogs function

* docs: explain for future self

* fix: another racecondition
also bulk insert auditLogs and auditLogTargets

* [autofix.ci] apply automated fixes

---------

Co-authored-by: chronark <dev@chronark.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
chronark added a commit that referenced this issue Oct 28, 2024
* fix:breadcrum-cache-issue

* chore:rename-revalidateMytag

* refactor: move-revalidateTag

* refactor: extract tags into reusable utility

* feat: add template

* Update 3_follow_the_unkey_x_account.md (#2323)

* fix: capture uniqueness error and return it correctly (#2337)

* docs: getVerifications instead of get (#2329)

* Update 3_follow_the_unkey_x_account.md (#2324)

Followed unkey on Twitter

Co-authored-by: Andreas Thomas <dev@chronark.com>

* fix: transaction ...: in use: for query (#2278)

* fix: transaction ...: in use: for query
- Fixes #2197
- The error is caused by the cache revalidation happening in the background, so we have a racecondition with the other queries we are running in the insertGenericAuditLogs function

* docs: explain for future self

* fix: another racecondition
also bulk insert auditLogs and auditLogTargets

* [autofix.ci] apply automated fixes

---------

Co-authored-by: chronark <dev@chronark.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

* fix: handle undefined ratelimits

* Update 3_follow_the_unkey_x_account.md (#2338)

* fix: handle case when blogs array is empty

* [oss.gg] side quest : added another framework; hono on cloudflare workers. (#2345)

* fix: weird spacing in changelog by removing the extra gap (#2340)

* fix: weird spacing in changelog

* feat: formatted

* oss.gg: create a template for ratelimiting a oak server in deno with unkey (#2308)

Co-authored-by: Andreas Thomas <dev@chronark.com>

* follow unkey on X #2252 (#2315)

Co-authored-by: Your Name <you@example.com>

* Follow the Unkey X account: @unkeydev Complete! (#2332)

* Update 3_follow_the_unkey_x_account.md

* Update 3_follow_the_unkey_x_account.md

---------

Co-authored-by: Andreas Thomas <dev@chronark.com>

* fix(www): analytics bento code snippet is not readable (#2311)

- add hover effect to analytics bento removing gradient background
- make copy code button sticky to top-right corner
- impl independent vertical scrolls for language switcher and code editor

* Update 7_create_a_template.md

* Update 7_create_a_template.md

* chore(deps-dev): bump @content-collections/core from 0.6.2 to 0.7.2 (#2255)

Bumps @content-collections/core from 0.6.2 to 0.7.2.

---
updated-dependencies:
- dependency-name: "@content-collections/core"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: order audit logs by time, show latest on top (#2295)

* feat: add template

* feat: Unkey FastAPI boilerplate (#2307)

* feat: add template

* feat: follow unkey on X (#2357)

* Update 6_record_onboarding.md (#2301)

* fix: reset input fields in API delete dialog after reopening (#2316)

* fix: reset input fields in API delete dialog after reopening

* refactor: remove unnecessary useEffect and utilize existing onOpenChange function for dialog

* refactor: removed unused imports

* Update 3_follow_the_unkey_x_account.md (#2364)

Added name to the followed list.

Co-authored-by: Andreas Thomas <dev@chronark.com>

* ci: add label

* feat: add hover to input fields

* Added gaps and width for md (#2371)

* docs: update overview.mdx (#2384)

avaliable -> available

* oss.gg side quest 3_follow_the_unkey_x_account.md (#2399)

Followed the unkey x account

* feat: Following unkey acc on twitter #2407 (#2408)

* Feat: Followed Unkey on X

* Feat: Followed Unkey on Twitter

---------

Co-authored-by: Andreas Thomas <dev@chronark.com>

* resolve merge conflicts

* resolve merge conflicts

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: chronark <dev@chronark.com>
Co-authored-by: Emily Marie Ahtúnan <86572370+Emmarie-Ahtunan@users.noreply.github.com>
Co-authored-by: Harsh Shrikant Bhat <90265455+harshsbhat@users.noreply.github.com>
Co-authored-by: Aritra Sadhukhan <60315087+aritradevelops@users.noreply.github.com>
Co-authored-by: Flo <53355483+Flo4604@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Devang Rakholiya <116096508+Devang0907@users.noreply.github.com>
Co-authored-by: Prabin <42871240+prabincankod@users.noreply.github.com>
Co-authored-by: ZiaCodes <72739794+Khaan25@users.noreply.github.com>
Co-authored-by: Devansh Baghel <77718741+Devansh-Baghel@users.noreply.github.com>
Co-authored-by: Shreenarayan jaiswal <127131074+shreenarayan123@users.noreply.github.com>
Co-authored-by: Your Name <you@example.com>
Co-authored-by: Chirag Arora <chirag8023@gmail.com>
Co-authored-by: Nazar Poshtarenko <32395926+unrenamed@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Saketh Pavan Goti <44170829+Sakethpavan@users.noreply.github.com>
Co-authored-by: Teddy ASSIH <110066080+Ionfinisher@users.noreply.github.com>
Co-authored-by: Vardhaman Bhandari <97441447+Vardhaman619@users.noreply.github.com>
Co-authored-by: Atharva_404 <72994819+Atharva-3000@users.noreply.github.com>
Co-authored-by: Rishi <rishipatel0826@gmail.com>
Co-authored-by: Ikko Eltociear Ashimine <eltociear@gmail.com>
Co-authored-by: Ayan Mondal <ayanmondal1805@gmail.com>
Co-authored-by: Kunal Arya <105645118+kunalarya873@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants