Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documents with Kura Kapua connection tutorial #780

Merged
merged 1 commit into from
Sep 11, 2017

Conversation

LeoNerdoG
Copy link
Contributor

I have created document which describes Kura - Kapua connection procedure.
This is just first bigger draft as more and more people are asking how to
connect those two. I will enhance it as the time goes.
If somebody finds mistakes, please correct them.

This PR is "answer" to Issue #751 .

Signed-off-by: Leonardo Gaube [email protected]

I have created document which describes Kura - Kapua connection procedure.
This is just first bigger draft as more and more people are asking how to
connect those two.

Signed-off-by: Leonardo Gaube <[email protected]>
@codecov
Copy link

codecov bot commented Aug 22, 2017

Codecov Report

Merging #780 into develop will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop     #780      +/-   ##
=============================================
- Coverage      53.57%   53.56%   -0.01%     
  Complexity       478      478              
=============================================
  Files            877      877              
  Lines          20543    20543              
  Branches        1843     1843              
=============================================
- Hits           11005    11003       -2     
- Misses          8822     8824       +2     
  Partials         716      716
Impacted Files Coverage Δ Complexity Δ
.../broker/core/plugin/KapuaSecurityBrokerFilter.java 78.13% <0%> (-0.46%) 0% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb29911...4092ac6. Read the comment docs.

@LeoNerdoG
Copy link
Contributor Author

Hi @Coduz ,
can you (or somebody else) please review and merge this document as it contains info regarding Kura-Kapua connection? I saw the issues regarding this matter and a lot of forum questions so I hope this helps.

Regards,
Leonardo

@ctron
Copy link
Contributor

ctron commented Aug 31, 2017

Thanks for the PR. I only see one issue with this. We already have two locations which describe how to start Kapua using docker. Adding a third location would make it more error prone once we begin changing this.

I would suggest that we consolidate that. One section in the documentation should describe the process, others should only reference it.

What do you think?

@LeoNerdoG
Copy link
Contributor Author

Hi @ctron ,
I am aware of that fact, but I tried to put myself into ordinary user which just tries to link Kura and Kapua. I know, that we are doubling or tripling the same document but I just wanted to make a guide, which contains everything to set up the desired connection. This way I think it is easier for a beginner to link the two. I hope you understand my dilemma.
But yes, I agree that documentation should be revised and renewed as soon as possible, because competitive projects have much better documentation, guides and tutorials.

Regards,
Leo

@riccardomodanese
Copy link
Contributor

Hi all, I'mo going to merge this pr. I'm aware of the changes requested by @ctron to organize the guide in a better way (avoiding duplication and so on).
So I'm going to open a new issue to track the reorganization of this guide ( #802 ) .

Regards

@riccardomodanese riccardomodanese merged commit 190f70c into eclipse-kapua:develop Sep 11, 2017
@LeoNerdoG LeoNerdoG deleted the kuraKapuaDocs branch September 12, 2017 12:13
@afterski
Copy link

afterski commented Dec 9, 2017

I've tested and it seems to work fine using the default user in kapua.
The information I needed (missing account name) was:
topic.context.account-name: kapua-sys
username: kapua-broker
password: kapua-password

However serializing objects are not allowed in kapua (dec 2017).
http://activemq.apache.org/objectmessage.html

so to get data in the DB
-Dorg.apache.activemq.SERIALIZABLE_PACKAGES=*
needs to be added to
ACTIVEMQ_OPTS in kapua-broker

This is obviously not a production solution but it gets You started with development.

Cheers, Björn

@riccardomodanese
Copy link
Contributor

riccardomodanese commented Dec 11, 2017

Thank you for your feedback.
The same issue was reported by https://github.com/yutouhewo (#1112 (comment)).
I'm investigating on that since this issue should be solved long time ago.

Can you open a new issue to keep tracking it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants