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

Increase e2e test timeout #571

Merged
merged 1 commit into from
Oct 29, 2020
Merged

Increase e2e test timeout #571

merged 1 commit into from
Oct 29, 2020

Conversation

honnix
Copy link
Member

@honnix honnix commented Oct 27, 2020

Try 20 minutes timeout

@honnix
Copy link
Member Author

honnix commented Oct 28, 2020

@wild-endeavor GitHub actions doesn't have strong enough machine to run e2e test fast enough. Increasing the timeout seems to work, but not so great.

@katrogan
Copy link
Contributor

@honnix can we merge this in while we debug what caused the timeout?

@honnix
Copy link
Member Author

honnix commented Oct 29, 2020

Sure I'm totally ok. I thought we would do some investigation first. Please go ahead.

@honnix honnix changed the title [DON'T MERGE] Debug e2e test timeout issue Increase e2e test timeout Oct 29, 2020
@katrogan katrogan merged commit cb77e4a into master Oct 29, 2020
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Jul 24, 2023
* not stripping structure from literal types

Signed-off-by: Daniel Rammer <[email protected]>

* fixed new lint issues

Signed-off-by: Daniel Rammer <[email protected]>

---------

Signed-off-by: Daniel Rammer <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Aug 9, 2023
* not stripping structure from literal types

Signed-off-by: Daniel Rammer <[email protected]>

* fixed new lint issues

Signed-off-by: Daniel Rammer <[email protected]>

---------

Signed-off-by: Daniel Rammer <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Aug 21, 2023
* test

Signed-off-by: Kevin Su <[email protected]>

* Execution tags

Signed-off-by: Kevin Su <[email protected]>

* Add tags filter

Signed-off-by: Kevin Su <[email protected]>

* Add execution tags table

Signed-off-by: Kevin Su <[email protected]>

* update tests

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

* update

Signed-off-by: Kevin Su <[email protected]>

* update

Signed-off-by: Kevin Su <[email protected]>

* update migration

Signed-off-by: Kevin Su <[email protected]>

* lint

Signed-off-by: Kevin Su <[email protected]>

* use gorm size

Signed-off-by: Kevin Su <[email protected]>

* Add tests

Signed-off-by: Kevin Su <[email protected]>

* bump idl

Signed-off-by: Kevin Su <[email protected]>

* bump idl

Signed-off-by: Kevin Su <[email protected]>

* bump idl

Signed-off-by: Kevin Su <[email protected]>

* address comment

Signed-off-by: Kevin Su <[email protected]>

* Set tag_name column as unique

Signed-off-by: Kevin Su <[email protected]>

* Add integration tests

Signed-off-by: Kevin Su <[email protected]>

* Update the tests

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

* nit

Signed-off-by: Kevin Su <[email protected]>

* BeforeCreate

Signed-off-by: Kevin Su <[email protected]>

* Update migration ID

Signed-off-by: Kevin Su <[email protected]>

* lint

Signed-off-by: Kevin Su <[email protected]>

* bump idl

Signed-off-by: Kevin Su <[email protected]>

* update migration id

Signed-off-by: Kevin Su <[email protected]>

---------

Signed-off-by: Kevin Su <[email protected]>
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.

2 participants