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

AtlasMap is unable to find mapping #4846

Closed
lburgazzoli opened this issue Mar 12, 2019 · 5 comments
Closed

AtlasMap is unable to find mapping #4846

lburgazzoli opened this issue Mar 12, 2019 · 5 comments
Assignees
@lburgazzoli
Labels
cat/bug A bug which needs fixing group/camel-k group/integration Library used in the the integration runtimes prio/p0 The priority of a bug. p0 means blocking
Milestone
Sprint 43 (4/4)

Comments

@lburgazzoli
Copy link
Collaborator

This is because we do sanitize the resource name like: https://github.com/syndesisio/syndesis/blob/1.6.x/app/server/controller/src/main/java/io/syndesis/server/controller/integration/camelk/CamelKPublishHandler.java#L344
@lburgazzoli lburgazzoli added cat/bug A bug which needs fixing prio/p0 The priority of a bug. p0 means blocking group/integration Library used in the the integration runtimes group/camel-k labels Mar 12, 2019
@lburgazzoli lburgazzoli added this to the Sprint 43 (4/4) milestone Mar 12, 2019
@lburgazzoli lburgazzoli self-assigned this Mar 12, 2019
@pure-bot pure-bot bot added the notif/triage The issue needs triage. Applied automatically to all new issues. label Mar 12, 2019
lburgazzoli added a commit to lburgazzoli/syndesis that referenced this issue Mar 12, 2019
@lburgazzoli
AtlasMap is unable to find mapping syndesisio#4846
61065ad
@lburgazzoli lburgazzoli mentioned this issue Mar 12, 2019
Merged
@heiko-braun
Copy link
Collaborator

How is this different from #4840?

@lburgazzoli
Copy link
Collaborator Author

The difference is that in #4840 the mapping is not attached to the mapping step whereas in this one, even when the mapping is present, atlasmap does not find it because the name is mangled

lburgazzoli added a commit to lburgazzoli/syndesis that referenced this issue Mar 12, 2019
@lburgazzoli
AtlasMap is unable to find mapping syndesisio#4846
0cea488
@heiko-braun
Copy link
Collaborator

@lburgazzoli Can you backport this to 1.6.x?

@lburgazzoli
Copy link
Collaborator Author

It is already on 1.6 61065ad

@heiko-braun heiko-braun removed notif/triage The issue needs triage. Applied automatically to all new issues. labels Mar 14, 2019
@tplevko
Copy link
Contributor

tplevko commented Apr 4, 2019

Camel-K TP related, QE ignore for now.

@tplevko tplevko closed this as completed Apr 4, 2019
@pure-bot pure-bot bot locked as resolved and limited conversation to collaborators Apr 4, 2019
@pure-bot pure-bot bot reopened this Apr 4, 2019
@pure-bot pure-bot bot unlocked this conversation Apr 4, 2019
@tplevko tplevko closed this as completed Apr 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@lburgazzoli lburgazzoli

Labels
cat/bug A bug which needs fixing group/camel-k group/integration Library used in the the integration runtimes prio/p0 The priority of a bug. p0 means blocking
Projects
None yet
Milestone
Sprint 43 (4/4)
Development

No branches or pull requests
3 participants
@heiko-braun @lburgazzoli @tplevko