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

ARTEMIS-3042 simple properties based, extensible broker image #4297

Merged
merged 1 commit into from
May 24, 2023

Conversation

gtully
Copy link
Contributor

@gtully gtully commented Nov 25, 2022

This module builds an OCI image tar, based around a broker Main class, that can be configured with property files and/or xml.
The intent is to be a simple as possible but still be useful. See the README.md for details.

@gtully gtully force-pushed the properties_based_broker_base_image branch from 65ff52f to adda78b Compare November 25, 2022 15:18
@gtully gtully force-pushed the properties_based_broker_base_image branch from adda78b to 8ee5d56 Compare January 25, 2023 15:23
artemis-image/README.md Outdated Show resolved Hide resolved
artemis-image/README.md Outdated Show resolved Hide resolved
artemis-image/README.md Outdated Show resolved Hide resolved
artemis-image/pom.xml Outdated Show resolved Hide resolved
artemis-image/pom.xml Outdated Show resolved Hide resolved
artemis-image/README.md Outdated Show resolved Hide resolved
artemis-image/README.md Outdated Show resolved Hide resolved
artemis-image/pom.xml Outdated Show resolved Hide resolved
artemis-image/pom.xml Outdated Show resolved Hide resolved
pom.xml Show resolved Hide resolved
artemis-image/pom.xml Show resolved Hide resolved
artemis-image/pom.xml Show resolved Hide resolved
@gtully gtully force-pushed the properties_based_broker_base_image branch 7 times, most recently from 91313c9 to e59f954 Compare May 15, 2023 12:58
@gtully
Copy link
Contributor Author

gtully commented May 15, 2023

externalised the base image in the fromImage property and updated to 20 for default

@gtully gtully force-pushed the properties_based_broker_base_image branch from e59f954 to 76aee8f Compare May 16, 2023 11:12
@gtully
Copy link
Contributor Author

gtully commented May 16, 2023

@tabish121 have you some cycles to take another peek, thanks in advance?

@gtully gtully requested review from tabish121 and gemmellr May 17, 2023 14:17
@tabish121
Copy link
Contributor

@tabish121 have you some cycles to take another peek, thanks in advance?

I've been looking at it and following through the docs and so far haven't stumbled on anything, looks good.

@gtully gtully force-pushed the properties_based_broker_base_image branch 2 times, most recently from f159d9a to 6b1b7f8 Compare May 19, 2023 10:50
@gtully gtully force-pushed the properties_based_broker_base_image branch from 6b1b7f8 to 8684c5c Compare May 19, 2023 12:16
@gtully gtully removed the request for review from tabish121 May 24, 2023 14:21
@gtully gtully merged commit 587ffb2 into apache:main May 24, 2023
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.

3 participants