Closed dervoeti closed 1 month ago
@siegfriedweber Thanks for the review so far, that was very helpful. I should have checked things more thoroughly, sorry, will do better next time.
I now also pinned the versions of s3fs
and cyclonedx-bom
to the current versions, they were not pinned before, which means the latest version gets installed (for example version 5.0.0 of cyclonedx-bom
came out 19 hours ago and got installed right away).
Sorry for the conflicts I caused.
When rebasing please make sure that there are no COPY or other instructions changing permissions after the chown
/chmod
combo. Feel free to ping me if you're unsure.
@lfrancke @siegfriedweber I rebased my changes on top of the current main branch now
Description
Fix needed for https://github.com/stackabletech/issues/issues/614
Changes:
RUN
commands to Heredocapache-airflow-providers-smtp
depends onapache-airflow
andapache-airflow
depends onapache-airflow-providers-smtp
. This causes the rendering of the dependency tree in SecObserve to fail.cyclonedx-gomod
(and I think Go in general) determines the version at build time. I did not find another way, I checked the source code ofcyclonedx-gomod
, that seems to be the only valid way. Since we don't include the.git
folder in our .tar.gz archives in Nexus, a dummy Git repo has to be created.