The patch in https://github.com/apache/incubator-druid/pull/8900 was effective.
If you have an open PR, please merge master into your branch in order to fix LGTM. On Mon, Nov 18, 2019 at 2:30 PM Gian Merlino <g...@apache.org> wrote: > Hey Druids, > > The LGTM tool is broken right now and it is holding up PRs from being > validated. Vadim and I looked into it a bit and it seems likely that > something is wrong with an internal TLS proxy on the LGTM side (certain > fetches to npm through that proxy don't work, even though they work outside > of LGTM). > > By the way, the particular step that is failing isn't providing any value. > It is building the web console during the Java validation step, which > doesn't verify the web console anyway. Ironically enough, the JavaScript > validation step works fine. > > We are trying to work around this problem using various different > techniques in https://github.com/apache/incubator-druid/pull/8900 and > https://github.com/apache/incubator-druid/pull/8897. If the issues > persist much longer I think it makes sense to ignore LGTM until it's fixed, > and merge things anyway, just so we don't get totally stuck. > > Gian >
