----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43072/#review117347 -----------------------------------------------------------
Bad patch! Reviews applied: [43072] Failed command: ./support/apply-review.sh -n -r 43072 Error: 2016-02-02 02:45:28 URL:https://reviews.apache.org/r/43072/diff/raw/ [226634/226634] -> "43072.patch" [1] Traceback (most recent call last): File "support/apply-reviews.py", line 342, in <module> reviewboard() File "support/apply-reviews.py", line 321, in reviewboard apply_review() File "support/apply-reviews.py", line 139, in apply_review commit_patch() File "support/apply-reviews.py", line 180, in commit_patch data = patch_data() File "support/apply-reviews.py", line 199, in patch_data return reviewboard_data() File "support/apply-reviews.py", line 252, in reviewboard_data email=user.get('email')) UnicodeEncodeError: 'ascii' codec can't encode character u'\xf8' in position 11: ordinal not in range(128) Full log: https://builds.apache.org/job/mesos-reviewbot/11187/console - Mesos ReviewBot On Feb. 1, 2016, 11:36 p.m., Michael Lunøe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43072/ > ----------------------------------------------------------- > > (Updated Feb. 1, 2016, 11:36 p.m.) > > > Review request for mesos, Bernd Mathiske and Thomas Rampelberg. > > > Bugs: MESOS-2585 > https://issues.apache.org/jira/browse/MESOS-2585 > > > Repository: mesos > > > Description > ------- > > The solution to our problem is solved by updating the Bootstrap CSS file, as > container-fluid was introduced later in Bootstrap. > > I have updated to lastest stable release of Bootstrap v3.3.6. Looking through > the change log, the only thing that we are using in the Mesos UI and changed > is the "hide" class (deprecated), so instances of these have been replaced by > its successor ".hidden" class. > > > Diffs > ----- > > src/webui/master/static/browse.html 6b18056 > src/webui/master/static/css/bootstrap-3.0.3.min.css c547283 > src/webui/master/static/css/bootstrap-3.3.6.min.css PRE-CREATION > src/webui/master/static/framework.html 9b28820 > src/webui/master/static/home.html d6cde1e > src/webui/master/static/index.html 25caf53 > src/webui/master/static/slave.html bc46885 > src/webui/master/static/slave_executor.html 9d582d5 > src/webui/master/static/slave_framework.html 96d788f > > Diff: https://reviews.apache.org/r/43072/diff/ > > > Testing > ------- > > - Mobile: http://cl.ly/2R2O0m1a1G3Q/Image%202016-02-01%20at%2015.21.53.png > - 1680 x 1050px screen: > http://cl.ly/0K1o112u3L2I/Image%202016-02-01%20at%2015.23.37.png > - 2880 x 1800px screen: > http://cl.ly/1M0L3M2X0J2i/Image%202016-02-01%20at%2015.22.50.png > > Visual test in latest Chrome, Firefox, Safari and IE. No functional changes > introduced. > > > Thanks, > > Michael Lunøe > >
