----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32541/#review78260 -----------------------------------------------------------
Master (3ab8a9e) is red with this patch. ./build-support/jenkins/build.sh :check :build :api:assemble :api:compileTestJava UP-TO-DATE :api:processTestResources UP-TO-DATE :api:testClasses UP-TO-DATE :api:test UP-TO-DATE :api:check UP-TO-DATE :api:build :buildSrc:compileJava UP-TO-DATE :buildSrc:processResources UP-TO-DATE :buildSrc:classes UP-TO-DATE :buildSrc:jar :buildSrc:assemble :buildSrc:compileTestJava UP-TO-DATE :buildSrc:processTestResources UP-TO-DATE :buildSrc:testClasses UP-TO-DATE :buildSrc:test UP-TO-DATE :buildSrc:check UP-TO-DATE :buildSrc:build BUILD SUCCESSFUL Total time: 4 mins 2.359 secs + export PIP_DEFAULT_TIMEOUT=60 + PIP_DEFAULT_TIMEOUT=60 + mkdir -p third_party + pip install -d third_party -r /dev/fd/63 ++ grep -v mesos.native 3rdparty/python/requirements.txt Downloading/unpacking bottle==0.11.6 (from -r /dev/fd/63 (line 1)) Saved ./third_party/bottle-0.11.6-py2.py3-none-any.whl Downloading/unpacking CherryPy==3.6.0 (from -r /dev/fd/63 (line 2)) pip can't proceed with requirement 'CherryPy==3.6.0 (from -r /dev/fd/63 (line 2))' due to a pre-existing build directory. location: /tmp/user/2395/pip_build_jenkins/CherryPy This is likely due to a previous installation that failed. pip is being responsible and not assuming it can delete this. Please delete it and try again. Storing debug log for failure in /home/jenkins/.pip/pip.log I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On March 30, 2015, 9:09 p.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32541/ > ----------------------------------------------------------- > > (Updated March 30, 2015, 9:09 p.m.) > > > Review request for Aurora, Kevin Sweeney and Brian Wickman. > > > Bugs: AURORA-813 > https://issues.apache.org/jira/browse/AURORA-813 > > > Repository: aurora > > > Description > ------- > > First take on client kerberos support. The idea is to repurpose the existing > auth_module system to support both legacy and kerberos during the deprecation > period. This way the 0.8.0 client will be able to talk to pre-0.8.0 scheduler > and use SessionKey-based authorization. Later (in 0.9.0), the payload() will > be removed along with SessionKey (AURORA-1229). That will let us get rid of > SchedulerProxy (or reduce it substantially). The auth_module might stay > though to support other auth plugins (e.g. requests-ntlm or > requests-oauthlib). > > TODO: integration e2e tests once scheduler side lands. > > > Diffs > ----- > > 3rdparty/python/requirements.txt 11a307cdb476ebcc25ab5c6b555bed29241ea988 > src/main/python/apache/aurora/client/api/__init__.py > a81329f6f947bbea4001c3a521c1923410a51eab > src/main/python/apache/aurora/client/api/scheduler_client.py > 95e553427492407743dcac31d70f392a7c1bbc02 > src/main/python/apache/aurora/client/cli/BUILD > c6b4e8a09d1315cf5defee2155a6e0c697892a30 > src/main/python/apache/aurora/client/cli/client.py > 24516d114db1743cdf600c542a27fcf5b68053a0 > src/main/python/apache/aurora/common/auth/BUILD > 966484627dab90e7606f1fc638cd0e159aee3317 > src/main/python/apache/aurora/common/auth/__init__.py > 3119fd63d3dfa28f93f219b23030059580fed098 > src/main/python/apache/aurora/common/auth/auth_kerberos.py PRE-CREATION > src/main/python/apache/aurora/common/auth/auth_module.py > 5f4116ef4cfbc407e0c50dc938870fb14e2299b4 > src/main/python/apache/aurora/common/auth/auth_module_manager.py > 73a8e5cd51edf694b971cd2c298ff406aff8c6d7 > src/test/python/apache/aurora/client/api/test_scheduler_client.py > 0a6194831c332a96eab62b869c4e05cfa9def058 > > Diff: https://reviews.apache.org/r/32541/diff/ > > > Testing > ------- > > ./pants test.pytest --no-fast src/test/python:all > ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh > > > Thanks, > > Maxim Khutornenko > >
