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

Disable QuarkusCodestartBuildIT for now #24544

Merged

Conversation

aloubyansky
Copy link
Member

@aloubyansky aloubyansky commented Mar 24, 2022

The tests are using https://stage.code.quarkus.io/api which seems to be broken at the moment.

Edit: it's is not broken, just the data tested did change on https://stage.code.quarkus.io/api/extensions

@aloubyansky aloubyansky merged commit afba0c2 into quarkusio:main Mar 24, 2022
@quarkus-bot quarkus-bot bot added this to the 2.9 - main milestone Mar 24, 2022
@gsmet
Copy link
Member

gsmet commented Mar 24, 2022

#21166 (comment) :)

@ia3andy
Copy link
Contributor

ia3andy commented Mar 25, 2022

#21166 (comment) :)

@gsmet Low blow :) The codestart can rely on stage.code.quarkus.io (code.quarkus.io would fail the same), but the test shouldn't.. it should either be removed or use mocked data, I will have a look.

@ia3andy
Copy link
Contributor

ia3andy commented Mar 25, 2022

Ok, so the failure is actually expected because it relies on some data which have changed. I will provide a PR.

@ia3andy
Copy link
Contributor

ia3andy commented Mar 25, 2022

AND @gsmet you may put the "I told you so" back in your closet 😈😂

@ia3andy
Copy link
Contributor

ia3andy commented Mar 25, 2022

@aloubyansky https://stage.code.quarkus.io/api doesn't exist, so nothing it's not broken :) try https://stage.code.quarkus.io/api/extensions.

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.

4 participants