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

Protocol error during tests #5116

Closed
feloy opened this issue Oct 4, 2021 · 3 comments · Fixed by #5126
Closed

Protocol error during tests #5116

feloy opened this issue Oct 4, 2021 · 3 comments · Fixed by #5126
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@feloy
Copy link
Contributor

feloy commented Oct 4, 2021

/kind bug

How did you run odo exactly?

The error happens during integration tests, on different clusters (IBM Cloud, PSI)

Actual behavior

protocol error: received *http.http2GoAwayFrame before a SETTINGS frame
Get "https://registry.stage.devfile.io/index": connection error: PROTOCOL_ERROR
protocol error: received *http.http2GoAwayFrame before a SETTINGS frame
[odo]  ✗  Failed to pull stack nodejs from registry.stage.devfile.io/devfile-catalog/nodejs:latest with allowed media types [application/vnd.devfileio.devfile.layer.v1 image/png image/svg+xml application/vnd.devfileio.vsx.layer.v1.tar application/x-tar]: failed to do request: Head "https://registry.stage.devfile.io/v2/devfile-catalog/nodejs/manifests/latest": connection error: PROTOCOL_ERROR

Expected behavior

Any logs, error output, etc?

@openshift-ci openshift-ci bot added the kind/bug Categorizes issue or PR as related to a bug. label Oct 4, 2021
@kadel
Copy link
Member

kadel commented Oct 5, 2021

This looks it might be something with devfile registry instance or registry library.
Maybe @GeekArthur or @johnmcollier could know something about this.

@feloy
Copy link
Contributor Author

feloy commented Oct 5, 2021

@kadel In your opinion, what would be the effort (installation + maintenance) of having our own devfile registry on the test cluster?

@kadel
Copy link
Member

kadel commented Oct 5, 2021

@kadel In your opinion, what would be the effort (installation + maintenance) of having our own devfile registry on the test cluster?

I don't think it is worth it to host our own instance. But we could avoid using devfile registry in most of the tests.

We can modify our test to not use any registry and use devfile.yaml files that are part of our test fixtures instead.

The only tests that are meant to test registry integration would be using remote devfile registry. Theoretically it could be only one test that does that.

feloy added a commit to feloy/odo that referenced this issue Oct 8, 2021
feloy added a commit to feloy/odo that referenced this issue Oct 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
2 participants