Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

QuarkusGenerator: Extract base image from config or fallback to default. #1748

Merged
merged 5 commits into from
Nov 4, 2019

Conversation

juazugas
Copy link
Contributor

Allow specifying in the configuration for the Quarkus generator the base image of the container.

This change extracts the configured key from for the generator or reads the property fabric8.generator.from to use it as a base image for the container.

It also fallbacks to the default values in case any is provided.

@codecov
Copy link

codecov bot commented Oct 26, 2019

Codecov Report

Merging #1748 into master will increase coverage by 0.15%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1748      +/-   ##
============================================
+ Coverage     34.39%   34.55%   +0.15%     
- Complexity     1129     1135       +6     
============================================
  Files           190      191       +1     
  Lines         10829    10900      +71     
  Branches       1763     1767       +4     
============================================
+ Hits           3725     3766      +41     
- Misses         6661     6688      +27     
- Partials        443      446       +3

@rohanKanojia
Copy link
Member

@juazugas : Thanks a lot for this PR 👍 Could you please run a mvn -N license:format to fix license issues. Also, please add a line to CHANGELOG regarding this change.

@rohanKanojia rohanKanojia requested review from manusa and oscerd November 4, 2019 09:50
Copy link
Member

@manusa manusa left a comment

Choose a reason for hiding this comment

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

LGTM

@devang-gaur devang-gaur merged commit ea3f6e2 into fabric8io:master Nov 4, 2019
@rohanKanojia rohanKanojia added the jkube/pending The issue/PR has to be taken care of in JKube https://github.com/eclipse/jkube label Nov 5, 2019
@rohanKanojia rohanKanojia removed the jkube/pending The issue/PR has to be taken care of in JKube https://github.com/eclipse/jkube label Nov 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants