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

tests/lib/prepare: temporarily disable HTTP/2 in the tests #7232

Closed
wants to merge 1 commit into from

Conversation

bboozzoo
Copy link
Contributor

Because of HTTP2 issues we keep seeing during snap download, disable it to
unblock submissions to master.

Details provided here:
[1]. #7227
[2]. https://forum.snapcraft.io/t/snap-download-failures-with-protocol-error/12677

Because of HTTP2 issues we keep seeing during snap download, disable it to
unblock submissions to master.

Details provided here:
[1]. canonical#7227
[2]. https://forum.snapcraft.io/t/snap-download-failures-with-protocol-error/12677

Signed-off-by: Maciej Borzecki <maciej.zenon.borzecki@canonical.com>
@bboozzoo bboozzoo added the Simple 😃 A small PR which can be reviewed quickly label Aug 12, 2019
@mvo5
Copy link
Contributor

mvo5 commented Aug 12, 2019

I'm not sure this is the right way forward, I did a quick google search for PROTOCOL_ERROR and it seems like our users are hitting this for real (e.g. https://forum.manjaro.org/t/opera-snap-wont-install/54127).

@bboozzoo
Copy link
Contributor Author

Let's try the retry path first. We can always revisit this one.

@bboozzoo bboozzoo closed this Aug 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Simple 😃 A small PR which can be reviewed quickly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants