aurora-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dmclaugh...@apache.org
Subject aurora git commit: Use update_job instead of creating new config object when modifying. This avoids losing any state (e.g. metadata) attached to the config.
Date Thu, 04 Aug 2016 21:20:55 GMT
Repository: aurora
Updated Branches:
  refs/heads/master d0533d2c7 -> ec3591c81


Use update_job instead of creating new config object when modifying. This avoids losing any
state (e.g. metadata) attached to the config.

Reviewed at https://reviews.apache.org/r/50819/


Project: http://git-wip-us.apache.org/repos/asf/aurora/repo
Commit: http://git-wip-us.apache.org/repos/asf/aurora/commit/ec3591c8
Tree: http://git-wip-us.apache.org/repos/asf/aurora/tree/ec3591c8
Diff: http://git-wip-us.apache.org/repos/asf/aurora/diff/ec3591c8

Branch: refs/heads/master
Commit: ec3591c812b4f7a063b3f5ee54ac66a9a4c5048b
Parents: d0533d2
Author: David McLaughlin <david@dmclaughlin.com>
Authored: Thu Aug 4 14:20:15 2016 -0700
Committer: David McLaughlin <dmclaughlin@twitter.com>
Committed: Thu Aug 4 14:20:15 2016 -0700

----------------------------------------------------------------------
 src/main/python/apache/aurora/client/cli/context.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/aurora/blob/ec3591c8/src/main/python/apache/aurora/client/cli/context.py
----------------------------------------------------------------------
diff --git a/src/main/python/apache/aurora/client/cli/context.py b/src/main/python/apache/aurora/client/cli/context.py
index 7bbfb5e..f1a256a 100644
--- a/src/main/python/apache/aurora/client/cli/context.py
+++ b/src/main/python/apache/aurora/client/cli/context.py
@@ -140,7 +140,9 @@ class AuroraCommandContext(Context):
             t.name == task.tier),
           task.production))
       }
-    return AnnotatedAuroraConfig(config.raw()(**backfill_args))
+
+    config.update_job(config.raw()(**backfill_args))
+    return config
 
   def get_job_config_optional(self, jobkey, config_file):
     """Loads a job configuration if provided."""


Mime
View raw message