Attention is currently required from: pespin.

View Change

5 comments:

• File osmo_dia2gsup-master/Dockerfile:

  • Patch Set #2, Line 19: rebar3 escriptize

    Maybe copy the resulting `osmo-dia2gsup` to `/usr/local/bin` here, so that it's easier to call it?

  • Patch Set #2, Line 26: --config /data/osmo_dia2gsup.config

    You're passing the config file twice. Isn't the `ERL_FLAGS` enough?

• File ttcn3-dia2gsup-test/Makefile:

  • Patch Set #2, Line 1:

    ws

• File ttcn3-dia2gsup-test/jenkins.sh:

  • Patch Set #2, Line 6: debian-bookworm-erlang

    This looks like a dependency of `osmo_dia2gsup-$IMAGE_SUFFIX`, and should not be listed here if so. This script is not using this image directly.

  • Patch Set #2, Line 31: /bin/sh -c "ERL_FLAGS='-config /data/osmo_dia2gsup.config' /tmp/osmo_dia2gsup/_build/default/bin/osmo-dia2gsup --config /data/osmo_dia2gsup.config >/data/osmo_dia2gsup.log 2>&1"

    This line is already specified in `CMD` of the respective Dockerfile, why duplicating it here?

To view, visit change 34215. To unsubscribe, or for help writing mail filters, visit settings.