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

infra: increase timeout for porch e2e tests #4119

Closed

Conversation

justinsb
Copy link
Contributor

We are hitting the default 10 minute timeout, possibly just because
they take time. Set a 20 minute timeout to match other tests.

Signed-off-by: justinsb [email protected]

We are hitting the default 10 minute timeout, possibly just because
they take time.  Set a 20 minute timeout to match other tests.

Signed-off-by: justinsb <[email protected]>
@justinsb
Copy link
Contributor Author

             exitCode: 1
             items:
               - image: gcr.io/kpt-fn/starlark:v0.2.1
Error:      -    stderr: '[error] : fail: could not find httpbin deployment'
            +    stderr: |-
            +      Status: Image is up to date for gcr.io/kpt-fn/starlark:v0.2.1
Error:             +      [error] : fail: could not find httpbin deployment
                 exitCode: 1
                 results:
                   - message: 'fail: could not find httpbin deployment'
FAIL

Maybe we do need #4113 , but it isn't deterministic...

/retest

@johnbelamaric
Copy link
Contributor

Porch has been moved to nephio - you may want to disable these while we await code removal?

nephio-project/nephio#465

We have the e2e tests working there, but don't yet have them running as CI. But that's where all the new stuff is being done.

@efiacor
Copy link

efiacor commented May 17, 2024

@justinsb
Copy link
Contributor Author

justinsb commented Jun 7, 2024

@mortent did you want to remove the code here so people don't get confused?

@justinsb
Copy link
Contributor Author

justinsb commented Jun 7, 2024

Ooops, @mortent has pointed out to me he already did this (removed our copy of porch) in #4153

/close

@kptdev-robot kptdev-robot bot closed this Jun 7, 2024
@kptdev-robot kptdev-robot bot closed this Jun 7, 2024
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.

3 participants