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

[XP3.0] Microprofile default config and FP build #229

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

Conversation

jfdenise
Copy link
Contributor

  • cekit module needed to build XP3.0 galleon feature-pack.

@jfdenise jfdenise requested a review from luck3y March 30, 2021 16:43
luck3y
luck3y previously approved these changes Mar 30, 2021

<installation xmlns="urn:jboss:galleon:provisioning:3.0">
<!-- Use the image version, to avoid retrieving metadata.xml for un-released s2i FP-->
<feature-pack location="eap-s2i@maven(org.jboss.universe:s2i-universe)#23.0.0.Final">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is the version correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We used to use the upstream version. I agree that we could align with product version. This version is not documented, but still.


<installation xmlns="urn:jboss:galleon:provisioning:3.0">
<!-- Use the image version, to avoid retrieving metadata.xml for un-released s2i FP-->
<feature-pack location="eap-s2i@maven(org.jboss.universe:s2i-universe)#23.0.0.Final">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem

<modelVersion>4.0.0</modelVersion>
<groupId>org.jboss.eap.galleon.s2i</groupId>
<artifactId>eap-s2i-galleon-pack</artifactId>
<version>23.0.0.Final</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that answers my question but I'm still confused. Why do we have a EAP artifact that uses a version which seems related to WildFly?

@luck3y
Copy link
Collaborator

luck3y commented May 11, 2021

@jfdenise @jmesnil Is this one OK to merge, or are changes still pending?

@jfdenise
Copy link
Contributor Author

@luck3y I changed the FP version to 3.0.0.Final to match product.

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

Successfully merging this pull request may close these issues.

3 participants