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

Add mosquitto Kubernetes yamls to development branch #1398

Merged
merged 1 commit into from Aug 7, 2017

Conversation

Projects
None yet
7 participants
@singhpratyush
Member

singhpratyush commented Aug 5, 2017

Short description

Fixes #1395.

Add yamls for mosquitto and change configuration for server to connect to mqtt via port 1883,

I have:

  • There is a corresponding issue for this pull request.
  • Mentioned the Issue number in the pull request commit message Fixes #<number> commit message
  • There is only strictly only one commit per issue.

For the reviewers

I have:

  • Reviewed this pull request by an authorized contributor.
  • The reviewer is assigned to the pull request.
@codecov-io

This comment has been minimized.

codecov-io commented Aug 5, 2017

Codecov Report

Merging #1398 into development will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##             development   #1398      +/-   ##
================================================
- Coverage           9.08%   9.07%   -0.02%     
  Complexity           403     403              
================================================
  Files                203     203              
  Lines              17968   17968              
  Branches            3315    3315              
================================================
- Hits                1632    1630       -2     
- Misses             16015   16016       +1     
- Partials             321     322       +1
Impacted Files Coverage Δ Complexity Δ
src/org/loklak/harvester/TwitterScraper.java 36.09% <0%> (-0.3%) 37% <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 e896ee9...7659f63. Read the comment docs.

@singhpratyush

This comment has been minimized.

Member

singhpratyush commented Aug 6, 2017

@Achint08 @djmgit @hemantjadon @kavithaenair @SKrPl @vibhcool: I've tested the changes and they're working. Please review.

@Achint08

LGTM 👍

@djmgit

This comment has been minimized.

Member

djmgit commented Aug 6, 2017

@singhpratyush rebase!

@mariobehling mariobehling merged commit c8f361b into loklak:development Aug 7, 2017

3 of 4 checks passed

codecov/project 9.07% (-0.02%) compared to e896ee9
Details
codacy/pr Good work! A positive pull request.
Details
codecov/patch Coverage not affected when comparing e896ee9...7659f63
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

mariobehling added a commit that referenced this pull request Aug 13, 2017

Update Master branch (#1448)
* Fixes #1402: Revert changes to backend in config.properties (#1403)

* Fixes #1387: Remove TwitterTweet methods from MessageEntry Class (#1388)

* Fix #1404: Change backend to root.loklak.org for Kubernetes images (#1405)

* Fix #1395 - Add mosquitto Kubernetes yamls to development branch (#1398)

* Fixes #1408: Post id doesn't appear in when source set to cache in search endpoint (#1409)

* Fix #1411: Add protocol for MQTT URI in Kubernetes Dockerfile (#1412)

* Fix #1406: Add methods to publish multiple message on multiple channels (#1407)

* final version (#1418)

* Fix #1413: Add MQTT publish hook in Post object (#1414)

* Related #1224: Add YAMLs for master deployment (#1377)

* Fixes #1350: Additional information added for display in StatusService (#1428)

* Fix #1424: Add channels to publish a Tweet on (#1427)

* Refactoring try-finally block to try resource block (#1426)

* Fix #1421: Add Docker images for different deployments (#1423)

* Fixes #1416: Deprecated Apache Ant in bluemix one click (#1417)

* Fix #1435: Add configurations for api.loklak.org deployment (#1436)

* Add use documentation, fix bug in the code (#1440)

* Related to #1439: Enhances the api.loklak.org page (#1441)

* Fix #1419: Remove readiness probe and increase initial delay (#1420)

Also introduce a gap of 300 seconds between probes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment