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.

Sign up for GitHub

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

Jump to bottom

Issue 134,137: Fix BookKeeper errors #135

Merged
merged 10 commits into from
Mar 20, 2019
Merged

Issue 134,137: Fix BookKeeper errors #135

merged 10 commits into from
Mar 20, 2019

Conversation

adrianmo
Copy link
Contributor

@adrianmo adrianmo commented Mar 12, 2019 •
edited
Loading

Change log description

  • Store BK index directory in a PVC
  • Set Pod request and limit resources
  • Set JVM heap memory values
  • Revert BK_useHostNameAsBookieID to false

Purpose of the change

Fix #134
Fix #137

How to test the change

Build an operator image based on this branch and run Pravega system tests.

Signed-off-by: Adrián Moreno adrian@morenomartinez.com

@adrianmo
Store BK index directory in a PVC
f575c7d 
Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>
@adrianmo adrianmo changed the title Store BK index directory in a PVC Issue 134: Store BK index directory in a PVC Mar 12, 2019
@adrianmo
fix operator version to 0.2.1
0d1e4c4 
Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>
@adrianmo
Re-enable tests
691e852 
Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>
@adrianmo
Update example CRs
d27bb11 
Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>
@adrianmo
Use IP as bookie ID
8476d8a 
Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>
@adrianmo adrianmo self-assigned this Mar 13, 2019
@adrianmo
Copy link
Contributor Author

BookKeeper errors on Kubernetes

@adrianmo
Set BK id to IP address
994878d 
Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>
@adrianmo adrianmo changed the title Issue 134: Store BK index directory in a PVC Issue 134,137: Fix BookKeeper errors Mar 14, 2019
@adrianmo
Specify container resources for testing
a1d16c7 
Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>
@adrianmo
Set Xmx and Xms Java opts to SS and C
6c15e5f 
Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>
@adrianmo
Adjust CPU and memory limits
462bf16 
Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>
@adrianmo
Merge branch 'master' into issue-134-add-index-volume
3a09e4d 
* master:
  Run tests on GKE (#140)
  Document update: README.md (#133)
  Add service account role definitions [skip ci] (#136)
@adrianmo adrianmo merged commit f58ca1e into master Mar 20, 2019
@adrianmo adrianmo deleted the issue-134-add-index-volume branch March 20, 2019 13:17
adrianmo added a commit that referenced this pull request Mar 20, 2019
@adrianmo
Merge branch 'master' into set-pod-resources
213d449 
* master:
  Issue 134,137: Fix BookKeeper errors (#135)
adrianmo added a commit that referenced this pull request Mar 29, 2019
@adrianmo
Merge branch 'master' into issue-81-upgrade-feature
f375f29 
* master:
  Issue 130: Ability to specify container resources (#129)
  Issue 134,137: Fix BookKeeper errors (#135)
  Run tests on GKE (#140)
  Document update: README.md (#133)
  Add service account role definitions [skip ci] (#136)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tristan1900 Tristan1900 Tristan1900 approved these changes

Assignees

@adrianmo adrianmo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@adrianmo @Tristan1900