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

Add a global hostname, pullPolicy and podAnnotations properties to chart #2073

Merged
merged 1 commit into from
Jun 7, 2021

Conversation

steven-sheehy
Copy link
Member

@steven-sheehy steven-sheehy commented Jun 4, 2021

Detailed description:

  • Add an importer.addressBook property that takes a Base64 encoded bootstrap address book to simplify non-prod configuration
  • Add a global.hostname property to simplify ingress configuration
  • Add a global.image.pullPolicy property
  • Add a global.podAnnotations property
  • Add a podAnnotations property to each component
  • Fix the monitor dashboard E2E subscriber single stat including REST latency
  • Fix the no pods ready alert triggering when any single pod went unready

Which issue(s) this PR fixes:

Special notes for your reviewer:
A follow up PR will use the global.podAnnotations and global.image.pullPolicy to automate deployments to integration.

Checklist

  • Documentation added
  • Tests updated

Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
@steven-sheehy steven-sheehy added enhancement Type: New feature P2 process Build or test related tasks labels Jun 4, 2021
@steven-sheehy steven-sheehy added this to the Mirror 0.36.0 milestone Jun 4, 2021
@steven-sheehy steven-sheehy requested a review from a team June 4, 2021 21:07
@steven-sheehy steven-sheehy self-assigned this Jun 4, 2021
@codecov
Copy link

codecov bot commented Jun 4, 2021

Codecov Report

Merging #2073 (e02d750) into master (126751e) will decrease coverage by 7.59%.
The diff coverage is 87.54%.

Impacted file tree graph

@@             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     
Impacted Files Coverage Δ
...ensus/ConsensusCreateTopicTransactionSupplier.java 0.00% <ø> (ø)
...sus/ConsensusSubmitMessageTransactionSupplier.java 0.00% <0.00%> (ø)
...ensus/ConsensusUpdateTopicTransactionSupplier.java 0.00% <ø> (ø)
...er/schedule/ScheduleCreateTransactionSupplier.java 0.00% <ø> (ø)
...supplier/token/TokenCreateTransactionSupplier.java 0.00% <ø> (ø)
...supplier/token/TokenUpdateTransactionSupplier.java 0.00% <ø> (ø)
...ra/mirror/monitor/config/MonitorConfiguration.java 0.00% <0.00%> (ø)
...edera/mirror/monitor/publish/PublishException.java 0.00% <0.00%> (ø)
.../hedera/mirror/monitor/publish/PublishMetrics.java 0.00% <0.00%> (ø)
...a/mirror/monitor/publish/TransactionPublisher.java 1.09% <0.00%> (-0.04%) ⬇️
... and 181 more

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 f70575c...e02d750. Read the comment docs.

@sonarcloud
Copy link

sonarcloud bot commented Jun 4, 2021

Copy link
Contributor

@ijungmann ijungmann left a 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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit.

Suggested change
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

Copy link
Member Author

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

charts/README.md Show resolved Hide resolved
charts/README.md Show resolved Hide resolved
Copy link
Contributor

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@steven-sheehy steven-sheehy merged commit b66af39 into master Jun 7, 2021
@steven-sheehy steven-sheehy deleted the podannotations branch June 7, 2021 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type: New feature P2 process Build or test related tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants