You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2020/09/02 09:41:25 UTC

[GitHub] [camel-kafka-connector] orpiske opened a new issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

orpiske opened a new issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409


   When trying to run the `camel-azure-storage-queue-kafka-connector` setting the properties directly (ie.: not via the URL), the code seems to fail to do so and fails with:
   
   ```org.apache.camel.ResolveEndpointFailedException: Failed to resolve endpoint: azure-storage-queue://test-queue23?accountName=devstoreaccount1 due to: Failed to resolve endpoint: azure-storage-queue://test-queue23?accountName=devstoreaccount1 due to: There are 1 parameters that couldn't be set on the endpoint. Check the uri if the parameters are spelt correctly and that they are properties of the endpoint. Unknown parameters=[{accountName=devstoreaccount1}]``` 
   
   I'll provide a reproducer along w/ the integration test for this one.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] omarsmak commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
omarsmak commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-685542452


   Sorry @orpiske, the docs were not clear in some parts, the correct url should be like this :
   `azure-storage-queue:/accountName/queueName?options`. If you get it working, please don't hesitate to correct the docs as I am currently in vacation :) 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] oscerd commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
oscerd commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-685518574


   cc @omarsmak 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] orpiske commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
orpiske commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-685583975


   Thanks @oscerd and @omarsmak. 
   
   TBH, I think the problem is on the way we are auto-generating this URL. To explain a bit further: I have 2 scenarios on the test. In one of them, the URL is setup manually (ie.: using `camel.sink.url`) and it works (the URL used is pretty much the same as the one you showed). On the other scenario, where I set everything via properties (ie.: `camel.sink.path.accountName`, `camel.sink.path.queueName`, etc), then it appears it creates the URL incorrectly.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] orpiske commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
orpiske commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-686335822


   @oscerd done :) 
   
   - this one is for the camel-azure-storage-queue: https://issues.apache.org/jira/browse/CAMEL-15499
   - this one is for the camel-azure-storage-blob, which seems to suffer from the same problem, but I haven't validated it: https://issues.apache.org/jira/browse/CAMEL-15500 
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] oscerd closed issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
oscerd closed issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] omarsmak commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
omarsmak commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-688245325


   I see it fixed, isn't @orpiske @oscerd ? Thanks for the PRs :) 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] orpiske commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
orpiske commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-686327208


   So, continuing the investigation on this one, it seems to me that this is the cause for the wrong URI: 
   
   - [azure-storage-queue.json](
   https://github.com/apache/camel/blob/master/components/camel-azure-storage-queue/src/generated/resources/org/apache/camel/component/azure/storage/queue/azure-storage-queue.json#L17 ) from master (3.5)
   - [azure-storage-queue.json](
   https://github.com/apache/camel/blob/camel-3.4.x/components/camel-azure-storage-queue/src/generated/resources/org/apache/camel/component/azure/storage/queue/azure-storage-queue.json#L17) from camel-3.4.x
   
   Since this (seems) to be fed into the [AbstractCamelCatalog.doAsEndpointUri](https://github.com/apache/camel/blob/camel-3.4.x/core/camel-core-catalog/src/main/java/org/apache/camel/catalog/impl/AbstractCamelCatalog.java#L695) which is called - at one point - as part of the [TaskHelper.buildUrl](https://github.com/apache/camel-kafka-connector/blob/master/core/src/main/java/org/apache/camel/kafkaconnector/utils/TaskHelper.java#L42).
   
   I am not sure, yet, how this one is generated. 
   
   
   
   
   
   
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] oscerd commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
oscerd commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-702522653


   Fixed. It will be in 3.6.x


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] orpiske commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
orpiske commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-686329371


   > Can you open an upstream issue?
   
   Absolutely. Just collecting the last bit of information here and I'll open one for this.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] oscerd commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
oscerd commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-686328775


   Can you open an upstream issue?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] orpiske commented on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
orpiske commented on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-686327876


   For the record, I think the format of the syntax in that json file should be `azure-storage-queue:/accountName/queueName`. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [camel-kafka-connector] orpiske edited a comment on issue #409: camel-azure-storage-queue-kafka-connector: Unknown parameter 'accountName'

Posted by GitBox <gi...@apache.org>.
orpiske edited a comment on issue #409:
URL: https://github.com/apache/camel-kafka-connector/issues/409#issuecomment-686335822


   @oscerd done :) 
   
   - this one is for the camel-azure-storage-queue: https://issues.apache.org/jira/browse/CAMEL-15499
   - this one is for the camel-azure-storage-blob, which seems to suffer from the same problem, but I did not verify it since I we don't have a TC for that yet: https://issues.apache.org/jira/browse/CAMEL-15500 
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org