-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change: Replace build.Dockerfile with build-openvas-smb-action compos… #90
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Call me a nitpicker ... or a inch pincher :D ^^
679bdb6
to
a606654
Compare
Co-authored-by: Jaspar Stach <jaspar.stach@greenbone.net>
Co-authored-by: Jaspar Stach <jaspar.stach@greenbone.net>
Co-authored-by: Jaspar Stach <jaspar.stach@greenbone.net>
Co-authored-by: Jaspar Stach <jaspar.stach@greenbone.net>
Co-authored-by: Jaspar Stach <jaspar.stach@greenbone.net>
18a8dee
to
ff9c3a9
Compare
gssapi seems to be deprecated in openvas-smb:
|
Unfortunately we are unable to change to a newer gssapi within openvas-smb. |
So, building the testing image fails also on the replaced workflow. |
9bde559
to
a32ca43
Compare
What will happen if we change to a newer version? |
We don't know the side effects, it is an very old samba fork without any kind of knowledge and very specific complexity. We may can get through if there was no function changes between the versions, but runtime behavior: no clue. That said we may can skip testing by saying in the moment testing gets stable openvas-smb must have been replaced by then. @ArnoStiefvater |
What
We decided to do some refactoring for openvas-smb.
openvas-smb-build build is deprecated.
But there are still workflows using that openvas-smb-build image.
Therefore we gonna replace the openvas-smb-build image with a manual step or composite action if used on multiple occasions.
Why
Removing build.Dockerfile and build-container.yml results in less code which is more maintainable.
References
https://jira.greenbone.net/browse/DEVOPS-1245
Checklist