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

Enrichers to configmap #1634

Closed
h3nrique opened this issue May 14, 2019 · 4 comments · Fixed by #1636
Closed

Enrichers to configmap #1634

h3nrique opened this issue May 14, 2019 · 4 comments · Fixed by #1636
Labels
cat/bug Bug which needs fixing cat/starter Easy issue, ideally for starting with fabric-maven-plugin

Comments

@h3nrique
Copy link
Contributor

Its not possible to change the name of configmap in tag configMap, always is xmlconfig.

$ oc get configmap
NAME        DATA      AGE
xmlconfig   1         20m

Description

Include in project the config below and run mvn fabric8:deploy

<configuration>
                            <resources>
                                <configMap>
                                    <entries>
                                        <entry>
                                            <file>src/main/resources/application.properties</file>
                                        </entry>
                                    </entries>
                                </configMap>
                            </resources>
                        </configuration>

Info

  • f-m-p version : 4.1.0
  • Maven version (mvn -v) :
Apache Maven 3.5.2
Maven home: /usr/share/maven
Java version: 1.8.0_171, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-openjdk-amd64/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "4.15.0-29deepin-generic", arch: "amd64", family: "unix"
  • Kubernetes / OpenShift setup and version (oc version) :
oc v3.10.45
kubernetes v1.10.0+b81c8f8
features: Basic-Auth GSSAPI Kerberos SPNEGO

Server https://192.168.42.210:8443
kubernetes v1.11.0+d4cacc0
  • Minishift version (minishift version) :
minishift v1.33.0+ba29431

Looking the classes [1] [2] I propose create a property name in class [2]. I will try create a local version and if works I open a pull request.

[1] - https://github.com/fabric8io/fabric8-maven-plugin/blob/master/enricher/standard/src/main/java/io/fabric8/maven/enricher/standard/ConfigMapEnricher.java
[2] - https://github.com/fabric8io/fabric8-maven-plugin/blob/master/core/src/main/java/io/fabric8/maven/core/config/ConfigMap.java

@rohanKanojia rohanKanojia added the cat/bug Bug which needs fixing label May 14, 2019
@rohanKanojia
Copy link
Member

@h3nrique : Yeah, looks like a bug which we should fix.

@rohanKanojia rohanKanojia added the cat/starter Easy issue, ideally for starting with fabric-maven-plugin label May 14, 2019
@rohanKanojia
Copy link
Member

This is due to hardcoding of name in ConfigMapEnricher here:

This class is used as model for ConfigMap XML configuration: ConfigMapEntry.java

@h3nrique
Copy link
Contributor Author

@rohanKanojia I can try update here, what a branch I can make a pull request?

@rohanKanojia
Copy link
Member

You can make a pull request against latest master

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cat/bug Bug which needs fixing cat/starter Easy issue, ideally for starting with fabric-maven-plugin
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants