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

Should be able to add custom route, even if fabric8.openshift.generateRoute=false #1720

Closed
erikgb opened this issue Oct 3, 2019 · 2 comments · Fixed by #1721
Closed

Should be able to add custom route, even if fabric8.openshift.generateRoute=false #1720

erikgb opened this issue Oct 3, 2019 · 2 comments · Fixed by #1721
Assignees

Comments

@erikgb
Copy link
Contributor

erikgb commented Oct 3, 2019

Description

We have decided to take more control over the Openshift routes for applications deployed with f-m-p. The route name is included in the URL to access the application from outside Openshift, and we need more control over the URLs for external access.

But when I set the fabric8.openshift.generateRoute to false (a sensible default in our case), to not generate the default route, no routes are added at all!!! This must be a bug.

To reproduce:

  • Disable route generation (fabric8.openshift.generateRoute=false)
  • Add a custom Route fragment
  • Run mvn fabric8:resource

Actual behavior: Your custom route is missing from ./target/classes/META-INF/fabric8
Expected behavior: An enriched version of your custom route in ./target/classes/META-INF/fabric8

I am pretty sure the cause of the this bug is the "catch all" solution implemented here:

Info

  • f-m-p version : 4.2.0
@erikgb
Copy link
Contributor Author

erikgb commented Oct 3, 2019

@rohanKanojia : I think this is a quite embarrassing bug that should be prioritized for the upcoming release. What do you think? At least I do not expect all my routes to be filtered out even if I set the property fabric8.openshift.generateRoute to false....

@rohanKanojia
Copy link
Member

Yeah, right. I think we should fix it. Let me fix it.

@rohanKanojia rohanKanojia self-assigned this Oct 3, 2019
rohanKanojia added a commit to rohanKanojia/fabric8-maven-plugin that referenced this issue Oct 3, 2019
…c8.openshift.generateRoute=false

Removed the filtering of route from ResourceMojo. Ideally it should go into RouteEnricher
rohanKanojia added a commit to rohanKanojia/fabric8-maven-plugin that referenced this issue Oct 4, 2019
…c8.openshift.generateRoute=false

Removed the filtering of route from ResourceMojo. Ideally it should go into RouteEnricher
rohanKanojia added a commit that referenced this issue Oct 4, 2019
…ift.generateRoute=false

Removed the filtering of route from ResourceMojo. Ideally it should go into RouteEnricher
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants