Add a "default_value" to the Environment Object of Rockon conf file

[aremiaskfa]

So, the current Environment Object schema looks like this:

{
  "description": "<Detailed description. Eg: Login username for Syncthing UI>",
  "label": "Web-UI username",
  (optional)"index": <integer: 1 or above. order of this environment variable, if relevant>
}

And I propose the following schema:

{
  "description": "<Detailed description. Eg: Login username for Syncthing UI>",
  "label": "Web-UI username",
  (optional)"default_value": <string: arbitrary value. suggested default value of this environment variable, if relevant>
  (optional)"index": <integer: 1 or above. order of this environment variable, if relevant>
}

Why?
I am currently trying to add a new Rockon to the registry: https://github.com/mbentley/docker-omada-controller/tree/master#building-images

Here the author mentions some caveats when configuring.
For example: the ports in the Ports section must be the same as the ones in the Environment section.

If possible, I’d like to offer the end-user the default values, so that the user doesn’t need to get confused by the fact that port values need to be entered twice.

In this sshot, I have entered the values manually:

[stitch10925]

I like this one. This way you can provision Rockons with Environment variables by default. Some applications (for example: DroneIO) rely heavily on Environmental Variables to function.

[Flox]

Hi @aremiaskfa,
Thanks a lot for your detailed feedback, always a pleasure to see that .

@stitch10925, thank you for your feedback as well.

I too, agree with both of you and think that it would be an important improvement overall. This was also a feature brought up by @magicalyak that we unfortunately had to leave unattended for too long now ; see below:

add env default option for rockons

This issue is also somewhat related and linked to another improvement: being able to make env configuration optional…

Rockons: Ability to set environment objects as optional

(see also issue #2280)
… that originated from one of @Hooverdan’s ideas:

Default values for environment variables in RockOn definition

I am looking at possibly updating the Watchtower Rockon (which helps managing auto-updates of docker containers - even multi-container setups, so a manual stop/start action is not necessary in the RockOn dashboard to get the latest version of an underlying container). The first commit was done about 2 years ago. I am interested in switching it to the official container ( containrrr/ watchtower) as well as surfacing more of the additional options, which can be defined as environment variables (i…

A lot to improve here and a lot of backlog, unfortunately. Now that we’re getting closer to having our next Stable release situated, though (see https://github.com/rockstor/rockstor-core/milestone/19), I am hopeful we will be able to spend more effort on such changes that require some database alteration(s).

Let us know if any of you is interested in having a go at those and we’ll do all we can to assist/support along the way, of course!