Avoid Using String in atomix config map about onos-test HOT 3 CLOSED

@Andrea-Campanella
I think it is already addressed correct? Can we close it?

from onos-test.

@adibrastegarnia it hasn't been changed. The code @Andrea-Campanella is referring to is here now:
https://github.com/onosproject/onos-test/blob/master/pkg/onit/cluster/cli.go#L89-L94

However, we can't make it separate key/value pairs in a ConfigMap because separate key/value pairs in a CM are mounted as separate files in the file system. But this is writing the configuration file used by Atomix, so it needs to be a single key. The only related improvement we could do here is encode a struct to YAML rather than writing a string.

from onos-test.

@kuujo
Good to know. I will take a look.

from onos-test.