Skip to content
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

Refactor makefile, add ubuntu1804 and fix label bug #117

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stolsma
Copy link
Contributor

@stolsma stolsma commented Feb 23, 2022

Add Ubuntu 18.04 as checked base image.
Refactor build, push and export steps
Fix label bug where label data with comma is truncated

Signed-off-by: stolsma [email protected]

@stolsma stolsma force-pushed the refactor-makefile branch 4 times, most recently from 91b21a3 to ba95a08 Compare February 23, 2022 12:49
@mestery
Copy link
Contributor

mestery commented Feb 23, 2022

Ubuntu 18.04 Bionic Beaver will be supported until April of 2023. Not sure we really want to add support for it here. What do you think @stolsma?

@stolsma
Copy link
Contributor Author

stolsma commented Feb 23, 2022

If we don't do support we also have to remove all the 18.04 specific build scripting IMHO. I don't think we are that far to decide on that, so that's why I added it to the CI build step. Then we at least will know when 18.04 doesn't work.... Also its only one reference to add at the moment so no big deal to add😉

Add Ubuntu 18.04 as checked base image.
Refactor build, push and export steps
Fix label bug where label data with comma is truncated

Signed-off-by: stolsma <[email protected]>
@stolsma
Copy link
Contributor Author

stolsma commented Feb 23, 2022

Btw it should be ready for review after this last running check (I removed some debugging lines...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants