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

fix(harness): retry CompleteStackRun with less data on entity too large #242

Conversation

floreks
Copy link
Member

@floreks floreks commented Jul 19, 2024

My local test code

	err := plural.Stacks.WithClient(client).Complete("0190bad3-f81c-4629-a1a9-ea7d683c7187", &console.StackRunAttributes{
		Status: console.StackStatusSuccessful,
		State: &console.StackStateAttributes{
			Plan:  lo.ToPtr(generator.Strings.Random(30_000_000)),
			State: generator.Stacks.State(20_000),
		},
	})
	if err != nil {
		log.Fatal(err)
	}

Output

2024/07/19 14:44:59 got error, running modifier: {"networkErrors":{"code":413,"message":"Response body \"Request Entity Too Large\""},"graphqlErrors":null}
2024/07/19 14:44:59 dropping state
2024/07/19 14:45:01 got error, running modifier: {"networkErrors":{"code":413,"message":"Response body \"Request Entity Too Large\""},"graphqlErrors":null}
2024/07/19 14:45:01 dropping plan
Completed stack run

@floreks floreks added the bug Something isn't working label Jul 19, 2024
@floreks floreks self-assigned this Jul 19, 2024
Copy link

linear bot commented Jul 19, 2024

@floreks floreks changed the title fix(harness): retry with less data complete stack run on entity too large fix(harness): retry CompleteStackRun with less data on entity too large Jul 19, 2024
floreks and others added 5 commits July 19, 2024 17:20
…-error-code-is-returned' of github.com:pluralsh/deployment-operator into sebastian/prod-2439-retry-w-no-terraform-state-if-a-413-error-code-is-returned
…etry-w-no-terraform-state-if-a-413-error-code-is-returned
@floreks floreks merged commit 6d3feb3 into main Jul 24, 2024
32 checks passed
@floreks floreks deleted the sebastian/prod-2439-retry-w-no-terraform-state-if-a-413-error-code-is-returned branch July 24, 2024 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants