-
Notifications
You must be signed in to change notification settings - Fork 108
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 a global hostname, pullPolicy and podAnnotations properties to chart #2073
Conversation
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
Codecov Report
@@ Coverage Diff @@
## master #2073 +/- ##
============================================
- Coverage 87.05% 79.46% -7.60%
- Complexity 1744 1920 +176
============================================
Files 315 377 +62
Lines 7731 9304 +1573
Branches 740 879 +139
============================================
+ Hits 6730 7393 +663
- Misses 772 1649 +877
- Partials 229 262 +33
Continue to review full report at Codecov.
|
SonarCloud Quality Gate failed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, couple nits in the README.
|
||
When deploying in GCP, the following steps may be taken to use container-native load balancer through |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit.
When deploying in GCP, the following steps may be taken to use container-native load balancer through | |
When deploying in GCP, the following steps may be taken to use a container-native load balancer through |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will correct in my follow up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Detailed description:
importer.addressBook
property that takes a Base64 encoded bootstrap address book to simplify non-prod configurationglobal.hostname
property to simplify ingress configurationglobal.image.pullPolicy
propertyglobal.podAnnotations
propertypodAnnotations
property to each componentWhich issue(s) this PR fixes:
Special notes for your reviewer:
A follow up PR will use the
global.podAnnotations
andglobal.image.pullPolicy
to automate deployments to integration.Checklist