subject:"\[jira\] \[Commented\] $METRON\-1188$ Ambari global configuration management broken"

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-21 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16174693#comment-16174693
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user justinleet commented on the issue:

https://github.com/apache/metron/pull/760
  
Thanks for making the adjustments, @mmiklavc.   I'm +1 on this.  It's a 
great step forward and gives a good foundation for future improvements we want 
to make.


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-20 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16173363#comment-16173363
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user mmiklavc commented on the issue:

https://github.com/apache/metron/pull/760
  
@justinleet One of my earlier commits erroneously triggered the Github 
"outdated comment" feature on some of your requests for javadocs. I've gone 
back through and added them.


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-19 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16172107#comment-16172107
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user mmiklavc commented on the issue:

https://github.com/apache/metron/pull/760
  
A @JonZeolla fixing it now. Sorry about that - I missed one of the 
"patch_path -> patch_file" arg changes in the mpack.


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-19 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16172104#comment-16172104
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user JonZeolla commented on the issue:

https://github.com/apache/metron/pull/760
  
This is failing to spin up for me.  As a part of Metron Enrichment Start, 
during:

```
2017-09-19 18:05:54,844 - Execute['/usr/metron/0.4.1/bin/zk_load_configs.sh 
--zk_quorum node1:2181 --mode PATCH --config_type GLOBAL --patch_path 
/tmp/metron-global-config-patch.json'] {'path': ['/usr/jdk64/jdk1.8.0_77/bin']}
```

I get
```
Traceback (most recent call last):
  File 
"/var/lib/ambari-agent/cache/common-services/METRON/0.4.1/package/scripts/enrichment_master.py",
 line 117, in 
Enrichment().execute()
  File 
"/usr/lib/python2.6/site-packages/resource_management/libraries/script/script.py",
 line 280, in execute
method(env)
  File 
"/var/lib/ambari-agent/cache/common-services/METRON/0.4.1/package/scripts/enrichment_master.py",
 line 58, in start
self.configure(env)
  File 
"/var/lib/ambari-agent/cache/common-services/METRON/0.4.1/package/scripts/enrichment_master.py",
 line 50, in configure
metron_service.refresh_configs(params)
  File 
"/var/lib/ambari-agent/cache/common-services/METRON/0.4.1/package/scripts/metron_service.py",
 line 129, in refresh_configs
patch_global_config(params)
  File 
"/var/lib/ambari-agent/cache/common-services/METRON/0.4.1/package/scripts/metron_service.py",
 line 111, in patch_global_config
path=ambari_format("{java_home}/bin")
  File "/usr/lib/python2.6/site-packages/resource_management/core/base.py", 
line 155, in __init__
self.env.run()
  File 
"/usr/lib/python2.6/site-packages/resource_management/core/environment.py", 
line 160, in run
self.run_action(resource, action)
  File 
"/usr/lib/python2.6/site-packages/resource_management/core/environment.py", 
line 124, in run_action
provider_action()
  File 
"/usr/lib/python2.6/site-packages/resource_management/core/providers/system.py",
 line 273, in action_run
tries=self.resource.tries, try_sleep=self.resource.try_sleep)
  File 
"/usr/lib/python2.6/site-packages/resource_management/core/shell.py", line 70, 
in inner
result = function(command, **kwargs)
  File 
"/usr/lib/python2.6/site-packages/resource_management/core/shell.py", line 92, 
in checked_call
tries=tries, try_sleep=try_sleep)
  File 
"/usr/lib/python2.6/site-packages/resource_management/core/shell.py", line 140, 
in _call_wrapper
result = _call(command, **kwargs_copy)
  File 
"/usr/lib/python2.6/site-packages/resource_management/core/shell.py", line 293, 
in _call
raise ExecutionFailed(err_msg, code, out, err)
resource_management.core.exceptions.ExecutionFailed: Execution of 
'/usr/metron/0.4.1/bin/zk_load_configs.sh --zk_quorum node1:2181 --mode PATCH 
--config_type GLOBAL --patch_path /tmp/metron-global-config-patch.json' 
returned 255. Unable to parse args: --zk_quorum node1:2181 --mode PATCH 
--config_type GLOBAL --patch_path /tmp/metron-global-config-patch.json
org.apache.commons.cli.UnrecognizedOptionException: Unrecognized option: 
--patch_path
at org.apache.commons.cli.Parser.processOption(Parser.java:363)
at org.apache.commons.cli.Parser.parse(Parser.java:199)
at org.apache.commons.cli.Parser.parse(Parser.java:85)
at 
org.apache.metron.common.cli.ConfigurationManager$ConfigurationOptions.parse(ConfigurationManager.java:140)
at 
org.apache.metron.common.cli.ConfigurationManager.main(ConfigurationManager.java:352)
usage: configuration_manager
 -c,--config_type The configuration type: GLOBAL,
   PARSER, ENRICHMENT, INDEXING,
   PROFILER
 -f,--forceForce operation
 -h,--help Generate Help screen
 -i,--input_dir   The input directory containing
   the configuration files named
   like "$source.json"
 -m,--mode   The mode of operation: DUMP,
   PULL, PUSH, PATCH
 -n,--config_name The configuration name: bro,
   yaf, snort, squid, etc.
 -o,--output_dir  The output directory which will
   store the JSON configuration
   from Zookeeper
 -pf,--patch_file  Path to the patch file.
 -pk,--patch_keyThe key to modify

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-18 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170784#comment-16170784
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user mmiklavc commented on a diff in the pull request:

https://github.com/apache/metron/pull/760#discussion_r139550653
  
--- Diff: 
metron-platform/metron-common/src/main/java/org/apache/metron/common/cli/ConfigurationManager.java
 ---
@@ -130,29 +201,39 @@ public void pull(CuratorFramework client, String 
outFileStr, final boolean force
   public void visit(ConfigurationType configurationType, String name, 
String data) {
 File out = getFile(outputDir, configurationType, name);
 if (!out.exists() || force) {
-  if(!out.exists()) {
+  if (!out.exists()) {
 out.getParentFile().mkdirs();
   }
   try {
 Files.write(data, out, Charset.defaultCharset());
   } catch (IOException e) {
 throw new RuntimeException("Sorry, something bad happened 
writing the config to " + out.getAbsolutePath() + ": " + e.getMessage(), e);
   }
-}
-else if(out.exists() && !force) {
+} else if (out.exists() && !force) {
   throw new IllegalStateException("Unable to overwrite existing 
file (" + out.getAbsolutePath() + ") without the force flag (-f or --force) 
being set.");
 }
   }
 });
   }
 
   public void push(String inputDirStr, CuratorFramework client) throws 
Exception {
-  final File inputDir = new File(inputDirStr);
+final File inputDir = new File(inputDirStr);
--- End diff --

Latest commit addresses the input dir null check.


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-18 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170775#comment-16170775
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user justinleet commented on a diff in the pull request:

https://github.com/apache/metron/pull/760#discussion_r139547949
  
--- Diff: 
metron-platform/metron-common/src/main/java/org/apache/metron/common/cli/ConfigurationManager.java
 ---
@@ -130,29 +201,39 @@ public void pull(CuratorFramework client, String 
outFileStr, final boolean force
   public void visit(ConfigurationType configurationType, String name, 
String data) {
 File out = getFile(outputDir, configurationType, name);
 if (!out.exists() || force) {
-  if(!out.exists()) {
+  if (!out.exists()) {
 out.getParentFile().mkdirs();
   }
   try {
 Files.write(data, out, Charset.defaultCharset());
   } catch (IOException e) {
 throw new RuntimeException("Sorry, something bad happened 
writing the config to " + out.getAbsolutePath() + ": " + e.getMessage(), e);
   }
-}
-else if(out.exists() && !force) {
+} else if (out.exists() && !force) {
   throw new IllegalStateException("Unable to overwrite existing 
file (" + out.getAbsolutePath() + ") without the force flag (-f or --force) 
being set.");
 }
   }
 });
   }
 
   public void push(String inputDirStr, CuratorFramework client) throws 
Exception {
-  final File inputDir = new File(inputDirStr);
+final File inputDir = new File(inputDirStr);
--- End diff --

Yeah, I think we could definitely handle the scripts better in general.

I'm not opposed to just leaving this as-is, but it seems like a decent / 
small opportunity to make it more usable. Up to you.


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-18 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170650#comment-16170650
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user mmiklavc commented on the issue:

https://github.com/apache/metron/pull/760
  
@justinleet Looks like you found the missing -i option. Also, it's funny 
that I confirmed "pp" and still ended up typing "pf." While I'm in here, what 
do you think is better, pp or pf? I'm happy to change it.


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-18 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170624#comment-16170624
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user justinleet commented on the issue:

https://github.com/apache/metron/pull/760
  
I spun this up in full dev and it worked really well. I think it's a great 
step towards cleaning up our config management. I'm still looking over some of 
the review updates and discussion, but this is a great.


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-18 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170615#comment-16170615
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user justinleet commented on the issue:

https://github.com/apache/metron/pull/760
  
Couple notes on the test instructions for anyone trying this out.
* Missing the input dir flag on the PUSH instructions, just need -i:
`$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m PUSH -c GLOBAL -i 
$METRON_HOME/config/zookeeper`
`$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m PUSH -c PARSER -n bro 
-i $METRON_HOME/config/zookeeper`
* The flag for files isn't `-pf`, it's `-pp` for patch path


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-18 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170600#comment-16170600
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user justinleet commented on a diff in the pull request:

https://github.com/apache/metron/pull/760#discussion_r139526927
  
--- Diff: 
metron-platform/metron-common/src/main/java/org/apache/metron/common/cli/ConfigurationManager.java
 ---
@@ -130,29 +201,39 @@ public void pull(CuratorFramework client, String 
outFileStr, final boolean force
   public void visit(ConfigurationType configurationType, String name, 
String data) {
 File out = getFile(outputDir, configurationType, name);
 if (!out.exists() || force) {
-  if(!out.exists()) {
+  if (!out.exists()) {
 out.getParentFile().mkdirs();
   }
   try {
 Files.write(data, out, Charset.defaultCharset());
   } catch (IOException e) {
 throw new RuntimeException("Sorry, something bad happened 
writing the config to " + out.getAbsolutePath() + ": " + e.getMessage(), e);
   }
-}
-else if(out.exists() && !force) {
+} else if (out.exists() && !force) {
   throw new IllegalStateException("Unable to overwrite existing 
file (" + out.getAbsolutePath() + ") without the force flag (-f or --force) 
being set.");
 }
   }
 });
   }
 
   public void push(String inputDirStr, CuratorFramework client) throws 
Exception {
-  final File inputDir = new File(inputDirStr);
+final File inputDir = new File(inputDirStr);
--- End diff --

The examples in the test plan don't work directly because they NPE here.  I 
assume it's because inputDirStr is null here and we only check for validity 
after wrapping it in a `File`.  While you're in here, could you add a check 
(and meaningful exception) for the case where inputDirStr is null?

Error is:
```
[root@node1 zookeeper]# $METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER 
-m PUSH -c GLOBAL
Exception in thread "main" java.lang.NullPointerException
at java.io.File.(File.java:277)
at 
org.apache.metron.common.cli.ConfigurationManager.push(ConfigurationManager.java:230)
at 
org.apache.metron.common.cli.ConfigurationManager.run(ConfigurationManager.java:256)
at 
org.apache.metron.common.cli.ConfigurationManager.run(ConfigurationManager.java:242)
at 
org.apache.metron.common.cli.ConfigurationManager.main(ConfigurationManager.java:350)
```


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-18 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170483#comment-16170483
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user mmiklavc commented on the issue:

https://github.com/apache/metron/pull/760
  
I also ran this through testing with Kerberos enabled in full dev.


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-18 Thread ASF GitHub Bot (JIRA)

[
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170424#comment-16170424
]

ASF GitHub Bot commented on METRON-1188:

Github user mmiklavc commented on the issue:

https://github.com/apache/metron/pull/760

### Test Plan
1. Run up full dev and verify you have data flowing through to the indexes
2. Change global config options.
1. Go into the Ambari management UI and modify any of the following
config options. e.g. set the es_clustername property to "ivebeenchanged". These
are values that end up in global.json and will prompt for a component restart.
Restart the components.
- es.clustername
- es.ip
- es.date.format
- parser.error.topic
- update.hbase.table
- update.hbase.cf
- profiler.client.period.duration
- profiler.client.period.duration.units
2. Verify $METRON_HOME/config/zookeeper/global.json now has the new
value. For the example above, you would expect to see
es.clustername=ivebeenchanged.
3. Verify Zookeeper contains the new config.
`$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m DUMP -c
GLOBAL`
3. Test changing configs from the command line. The zk config utils has
been updated with new backwards compatible features, so we'll test that here.
1. Change one of the zookeeper configs locally and then try a vanilla
config PUSH using the original method. Verify the values made it into zookeeper.
`${METRON_HOME}/bin/zk_load_configs.sh -i
${METRON_HOME}/config/zookeeper -m PUSH -z $ZOOKEEPER`
2. Change a config locally and push JUST that config to Zookeeper by
using one of the new config loader options. You can then perform a DUMP in a
similar manner.
```
# global config only
$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m PUSH -c GLOBAL
# bro config only
$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m PUSH -c PARSER
-n bro
# take a dump of config
$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m DUMP -c GLOBAL
$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m DUMP -c PARSER
-n bro
```
3. Try the new JSON patch mechanism with key/value from the command
line. Below are some examples but please feel free to explore other
combinations. **Note** - commons cli strips the first outermost pair of quotes
from all arguments, so you have to provide escaped quotes twice. Neither
\""bar"\" nor "\"bar\"" will work. You must provide both escaped quotes for
string values. \"\"bar\"\". Complex objects are a bit easier, but you still
have to escape quotes inside the string to get them to parse properly.
```
# global config only - add a new key "foo" with simple string value
"bar"
$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m PATCH -c
GLOBAL -pm ADD -pk foo -pv \"\"bar\"\"
# global config only - add a new key "foo" to the JSON doc root
with complex value "{ \"bar\" : { \"baz\" : [ \"bazval1\", \"bazval2\" ] } }"
$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m PATCH -c
GLOBAL -pm ADD -pk "/foo" -pv "{ \"bar\" : { \"baz\" : [ \"bazval1\",
\"bazval2\" ] } }"
# global config only - remove a key altogether
$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m PATCH -c
GLOBAL -pm REMOVE -pk "/foo"
```
4. Try the new JSON patch mechanism with a patch file. Below are some
examples but please feel free to explore other combinations. Note that a patch
file is a JSON array of individual patches.
1. Try a single patch from file
```
# Create a single patch that adds a new node. Create file
"/tmp/mypatch.txt" and add the following JSON array.
[
{
"op": "add",
"path": "/you",
"value": { "cannot" : { "handle" : [ "the", "flow" ] } }
}
]

# Perform the patch
$METRON_HOME/bin/zk_load_configs.sh -z $ZOOKEEPER -m PATCH -c GLOBAL
-pf /tmp/mypatch.txt

# Should result in adding the following to global.json
"you" : {
"cannot" : {
"handle" : [ "the", "flow" ]
}
}
```

2. Try multiple patches in a patch file
```
Create a single patch file that adds multiple nodes. Create file
"/tmp/mygrimespatch.txt" and add the following JSON array.
# 2 patches in a single patch file
[
{
"op": "add",
"path": "/la",
"value": { "jiggy" : { "jar" : { "jar" : "do" } } }

[jira] [Commented] (METRON-1188) Ambari global configuration management broken

2017-09-16 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/METRON-1188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16168835#comment-16168835
 ] 

ASF GitHub Bot commented on METRON-1188:


Github user mmiklavc closed the pull request at:

https://github.com/apache/metron/pull/760


> Ambari global configuration management broken
> -
>
> Key: METRON-1188
> URL: https://issues.apache.org/jira/browse/METRON-1188
> Project: Metron
>  Issue Type: Bug
>Reporter: Michael Miklavcic
>Assignee: Michael Miklavcic
>
> Ambari currently ignores values from the UI when managing the global.json 
> file. These values are hard-coded in the status_params.py file as part of the 
> mpack. In addition, Ambari restarts are clobbering configuration changes in 
> Zookeeper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

36 matches

Mail list logo