Skip to content
This repository has been archived by the owner on Mar 14, 2023. It is now read-only.

Add qatzip compressor plugin and run check_format.py #60

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

rojkov
Copy link
Contributor

@rojkov rojkov commented Feb 1, 2021

This PR supersedes #57 or can be just merged after it.

@ipuustin
Copy link
Contributor

Tried to test this but it doesn't seem to compile for me... @rojkov does it work for you?

$ docker image build -t envoy-qat:devel -f Dockerfile.openssl .

... ... ... ...

Analyzing: target //:envoy (632 packages loaded, 19812 targets configured)
INFO: Repository org_unicode_icuuc instantiated at:
  no stack (--record_rule_instantiation_callstack not enabled)
Repository rule http_archive defined at:
  /build/dbf569689a31dc8415b28f18f9ea16b0/external/bazel_tools/tools/build_defs/repo/http.bzl:336:31: in <toplevel>
WARNING: Download from https://github.com/unicode-org/icu/archive/release-64-2.tar.gz failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException Checksum was 65271a83fa81783d1272553f4564965ac2e32535a58b0b8141e9f4003afb0e3a but wanted 524960ac99d086cdb6988d2a92fc163436fd3c6ec0a84c475c6382fbf989be05
ERROR: An error occurred during the fetch of repository 'org_unicode_icuuc':
   java.io.IOException: Error downloading [https://github.com/unicode-org/icu/archive/release-64-2.tar.gz] to /build/dbf569689a31dc8415b28f18f9ea16b0/external/org_unicode_icuuc/release-64-2.tar.gz: Checksum was 65271a83fa81783d1272553f4564965ac2e32535a58b0b8141e9f4003afb0e3a but wanted 524960ac99d086cdb6988d2a92fc163436fd3c6ec0a84c475c6382fbf989be05
ERROR: /build/dbf569689a31dc8415b28f18f9ea16b0/external/com_googlesource_googleurl/url/BUILD:6:11: @com_googlesource_googleurl//url:url depends on @org_unicode_icuuc//:common in repository @org_unicode_icuuc which failed to fetch. no such package '@org_unicode_icuuc//': java.io.IOException: Error downloading [https://github.com/unicode-org/icu/archive/release-64-2.tar.gz] to /build/dbf569689a31dc8415b28f18f9ea16b0/external/org_unicode_icuuc/release-64-2.tar.gz: Checksum was 65271a83fa81783d1272553f4564965ac2e32535a58b0b8141e9f4003afb0e3a but wanted 524960ac99d086cdb6988d2a92fc163436fd3c6ec0a84c475c6382fbf989be05
ERROR: Analysis of target '//:envoy' failed; build aborted: Analysis failed
INFO: Elapsed time: 37.052s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (632 packages loaded, 19812 targets configured)

@rojkov
Copy link
Contributor Author

rojkov commented Feb 15, 2021

Checksum was 65271a83fa81783d1272553f4564965ac2e32535a58b0b8141e9f4003afb0e3a but wanted 524960ac99d086cdb6988d2a92fc163436fd3c6ec0a84c475c6382fbf989be05

No, doesn't work for me neither. I bet the master is broken too.

@mythi
Copy link
Contributor

mythi commented Feb 15, 2021

we have #55 to remind about it.

We should probably decouple qatzip filter from envoy-openssl...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants