----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64341/#review193801 -----------------------------------------------------------
I am traveling for the next 5 days, so my review will be delayed. Feel free to ship without me though. - Stephan Erb On Dec. 11, 2017, 12:13 p.m., Jing Chen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64341/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2017, 12:13 p.m.) > > > Review request for Aurora, David McLaughlin, Stephan Erb, and Bill Farner. > > > Bugs: AURORA-1898 > https://issues.apache.org/jira/browse/AURORA-1898 > > > Repository: aurora > > > Description > ------- > > Add metadata field to Job object in DSL > > > Diffs > ----- > > RELEASE-NOTES.md 54dcc75ef4f4fbf52c4dc60ec0dcbdd7ff3926a2 > docs/reference/configuration.md 67d9914f6b43f5eb73fe05b547981737f665489c > src/main/python/apache/aurora/config/schema/base.py > a466e78f85d980dc11689ab252b8c70e9cfd3d57 > src/main/python/apache/aurora/config/thrift.py > eb0014422f99fdc8dcea9b0b75b86cdf78cf2dbb > src/test/python/apache/aurora/client/cli/test_inspect.py > ecefc1845350242fe5113e5c85c4e3db09937ec1 > src/test/python/apache/aurora/config/test_thrift.py > 76d0ad681689e7aa03401ecea2d2b3123268d7da > > > Diff: https://reviews.apache.org/r/64341/diff/2/ > > > Testing > ------- > > Test with `.aurora` file as: > ``` > pkg_path = '/vagrant/hello.py' > > install = Process( > name='fetch package', > cmdline='cp {} . && chmod u+x hello.py'.format(pkg_path)) > > runner = Process( > name='hello world', > cmdline='python -u hello.py') > > hello_task = SequentialTask( > processes=[install, runner], > resources=Resources(cpu=1, ram=1*MB, disk=8*MB)) > > jobs = [ > Service(cluster='devcluster', > role='www-data', > environment='devel', > name='metadata_test', > task=hello_task, > metadata=[Metadata(key='state', value='CA'), Metadata(key='stat', > value='NY')] > ) > ] > ``` > > Metadata information can be found in **job configuration** of UI as: > ``` > METADATA > state: CA > stat: NY > ``` > > > Thanks, > > Jing Chen > >
