Resolve "Force `ddeploy` options (e.g. image retention)"

Closes #43 (closed)

assigned to @jon.azpiazu

added 1 commit

• 25ae5c49 - Add initial script to add the retention labels to existing image

Compare with previous version

added 1 commit

• 2c21e985 - Minor style fixes to script

Compare with previous version

I would call the script in this MR just before this line.

What do you think @miguel.prada @inigo.moreno ?

added 1 commit

• 4c1a0784 - Skip debug mode and reduce verbosity

Compare with previous version

added 1 commit

• 5ffba0f4 - Fix shellcheck complains

Compare with previous version

resolved all threads

added 1 commit

• 3518b83b - Add enforce labels to ddeploy job

Compare with previous version

added 2 commits

• 9c5cc8c9 - Add some debugging
• 10cdfb0e - Make sure bash is installed, as is needed by the enforce_labels script

Compare with previous version

added 1 commit

• 19433cc2 - Redo calling remote script to avoid sh problems

Compare with previous version

added 1 commit

• 11498a78 - Fix requirements

Compare with previous version

added 1 commit

• bb00a01f - Add missing dependency

Compare with previous version

Test job:

https://git.code.tecnalia.com/tecnalia_robotics/flexbotics/flexbotics_test/dummy_application/-/jobs/339771

requested review from @inigo.moreno

marked this merge request as ready

I don't fully understand the code in enforce_labels.bash, but it looks like it worked as intended, and the output on the CI job looks understandable.

I would suggest adding some comments to the bash file, otherwise LGTM.

approved this merge request

added 1 commit

• 58a0bce5 - Add comments to script

Compare with previous version

added 1 commit

• d3a1ed95 - Fix branch name before merging

Compare with previous version

resolved all threads

resolved all threads

merged

mentioned in commit c873c66a