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

[0.6.0] Error on upstream entities during sync #96

Closed
miguel-martinez-tfs opened this issue Nov 4, 2019 · 0 comments · Fixed by #97
Closed

[0.6.0] Error on upstream entities during sync #96

miguel-martinez-tfs opened this issue Nov 4, 2019 · 0 comments · Fixed by #97
Assignees
Labels
bug Something isn't working
Milestone

Comments

@miguel-martinez-tfs
Copy link

miguel-martinez-tfs commented Nov 4, 2019

Trying to sync a yaml file with multiple different upstreams (different ids, names and targets) on Deck 0.6.0 throws the following error on verbose:

GET /acls?size=1000 HTTP/1.1
Host: localhost:8001
User-Agent: Go-http-client/1.1
Accept-Encoding: gzip

HTTP/1.1 200 OK
Content-Length: 23
Access-Control-Allow-Origin: *
Connection: keep-alive
Content-Type: application/json; charset=utf-8
Date: Mon, 04 Nov 2019 19:20:04 GMT
Server: kong/0.35-1
X-Kong-Admin-Request-Id: b66HEmYXMu93vKXNQGhuPhZrACVBNMCk

{"next":null,"data":[]}
Error: inserting upstream into state: entity already exists

The error will not occur if only a single upstream is defined on the yaml file. A workaround exists by synching one upstream at a time, but I expect them to be synched all at once.

@hbagdi hbagdi self-assigned this Nov 4, 2019
@hbagdi hbagdi added the bug Something isn't working label Nov 4, 2019
@hbagdi hbagdi added this to the v0.6.1 milestone Nov 4, 2019
hbagdi added a commit that referenced this issue Nov 5, 2019
The bug here is de-referencing the loop variable, which gets updated
on every iteration and leads to all items being set to the last item in
the slice.

Fix #96
@hbagdi hbagdi mentioned this issue Nov 5, 2019
@hbagdi hbagdi closed this as completed in #97 Nov 5, 2019
hbagdi added a commit that referenced this issue Nov 5, 2019
The bug here is de-referencing the loop variable, which gets updated
on every iteration and leads to all items being set to the last item in
the slice.

Fix #96
rainest pushed a commit that referenced this issue Apr 21, 2021
The bug here is de-referencing the loop variable, which gets updated
on every iteration and leads to all items being set to the last item in
the slice.

Fix #96
AntoineJac pushed a commit that referenced this issue Jan 23, 2024
The bug here is de-referencing the loop variable, which gets updated
on every iteration and leads to all items being set to the last item in
the slice.

Fix #96
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants