Closed vincent-zurczak closed 7 years ago
Instead of adding a new parameter, I think we should simply replace the target-id parameter by a new one called parameters. This property could take the following values:
file:/
or http:/
.WDYT? @gibello @diarraa As this is quite important for Docker support, I would like quick feedback. :warning: The idea is that we should be able to pass user data to a Dockerized agent with a single command (e.g. a shared volume) instead of using N environment variables (see https://github.com/roboconf/roboconf-dockerfile).
Currently, we have to specify in the agent what the target is so that we can read user data. However, an agent is also able to read user data from a file. The location of this file may be different from one situation to another. So, we should add a new parameter in the agent's configuration to retrieve user data from a file.
This parameter will be used only when the target is none (which means it would not be read when we deploy over AWS or Openstack). See also roboconf/roboconf-dockerfile#10.