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

feat(trait): Add maven profile to Integration in builder trait #4568

Merged
merged 1 commit into from
Jul 26, 2023

Conversation

gansheer
Copy link
Contributor

Ref #4560

Description

  • Add builder.maven-profile property referencing a secret or a configmap : [configmap|secret]:name[/key]
  • Inject the given profile into the pom.xml generated in the project
  • Some refactoring on configmap/secret decoding from cli

Example:

  • Creates a secret or a configmap: kubectl create secret generic maven-profile-owasp --from-file=owasp-profile.xml
  • Use it in the builder trait: -t builder.maven-profile=secret:maven-profile-owasp/owasp-profile.xml

Release Note

Add maven profile to Integration in builder trait as the maven-profile property referencing a secret or a configmap

@github-actions
Copy link
Contributor

🐫 Thank you for contributing!

Code Coverage Report ✔️ - Coverage unchanged.

1 similar comment
@github-actions
Copy link
Contributor

🐫 Thank you for contributing!

Code Coverage Report ✔️ - Coverage unchanged.

Copy link
Contributor

@squakez squakez left a comment

Choose a reason for hiding this comment

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

Really cool feature, congrats! Just a little remark which I hope we can change to have the code completely separate from any external configuration.

pkg/builder/project.go Show resolved Hide resolved
@squakez squakez added the kind/feature New feature or request label Jul 13, 2023
@squakez squakez modified the milestone: 2.0.0 Jul 13, 2023
@github-actions
Copy link
Contributor

🐫 Thank you for contributing!

Code Coverage Report ✔️ - Coverage unchanged.

@github-actions
Copy link
Contributor

🐫 Thank you for contributing!

Code Coverage Report ✔️ - Coverage unchanged.

Ref apache#4560

* Add builder.maven-profile property referencing a secret or a configmap
* Inject the given profile into the pom.xml generated
* Some refactoring on configmap/secret decoding from cli
@github-actions
Copy link
Contributor

🐫 Thank you for contributing!

Code Coverage Report ✔️ - Coverage unchanged.

@squakez
Copy link
Contributor

squakez commented Jul 25, 2023

All green. Is there any other pending work or can we merge @gansheer?

@gansheer
Copy link
Contributor Author

It's complete, ready for merge 🎇 .

@squakez squakez merged commit b41287a into apache:main Jul 26, 2023
16 checks passed
@squakez
Copy link
Contributor

squakez commented Jul 26, 2023

Thanks for the contribution. Consider to add some documentation to show usage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants