This is fine, thank you for sending a note. In the future, when we are ready to make container releases along with Beam releases, we can have a cleaned up version of this container.
On Thu, Mar 21, 2019 at 9:36 AM Mikhail Gryzykhin <[email protected]> wrote: > Hi everyone, > > Recently, there was added a test for verifying metrics in python SDK ( > PR-8038 <https://github.com/apache/beam/pull/8038>). > > This PR causes beam_PostCommit_Py_ValCont job to fail > <https://builds.apache.org/job/beam_PostCommit_Py_ValCont/2683/consoleFull> > due to lack of pyhamcrest library in python SDK container. > > I have created a PR-8107 <https://github.com/apache/beam/pull/8107> that > adds relevant library and it fixes test. However I want to first confirm > that we are OK with adding library used for testing to prod container or > whether we want to utilize some other approach for testing. > > For a background: we already have tenacity library added this way. > > Regards, > --Mikhail > > Have feedback <http://go/migryz-feedback>? >
