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

Fog-Report - Helm chart improvements for consolidating fog instances. #3643

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

jgreat
Copy link
Contributor

@jgreat jgreat commented Oct 23, 2023

  • Break out fog-report chart from fog-services monolith.
  • Update GHA and integration tests to support multiple fog-report instances.

Motivation

We need to be able to build fog instances that have multiple copies of fog-report, fog-view-router and fog-ledger-router with appropriate ingress and secret alignment.

@jgreat jgreat changed the base branch from main to release/v5.1 October 23, 2023 23:36
@jgreat jgreat changed the title WIP: Helm chart improvements for consolidating fog instances. Fog-Report - Helm chart improvements for consolidating fog instances. Oct 24, 2023
@jgreat jgreat requested review from joekottke, a team and nick-mobilecoin and removed request for a team October 24, 2023 20:55
@jgreat jgreat marked this pull request as ready for review October 24, 2023 20:59
Copy link
Contributor

@joekottke joekottke left a comment

Choose a reason for hiding this comment

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

Just curious about the deletions, and a few comments. Non-blocking.

@jgreat jgreat merged commit 605f13d into release/v5.1 Oct 25, 2023
84 checks passed
@jgreat jgreat deleted the one-fog-charts branch October 25, 2023 15:38
aweis89 pushed a commit that referenced this pull request Oct 25, 2023
nick-mobilecoin added a commit that referenced this pull request Nov 10, 2023
* Fog ledger update prometheus metrics in main loop (#3649)

Previously fog ledger would only report the ledger db metrics on startup
and not update the values. Now fog ledger will continue to update the
prometheus metrics in its main loop

* break out fog-report chart (#3643)

* fog-view - Helm chart improvements for consolidating fog instances (#3662)

* break out fog-view chart

* bump grpcio to 0.13 (#3674)

* fog-ledger - Helm chart improvements for consolidating fog instances (#3668)

* break out ledger chart and deprecate fog-services

Co-authored-by: Nick Santana <[email protected]>

* Remove fog-services charts. (#3675)

* remove fog-services fog-service-config charts
* remove client-responder-id labels. The value can exceed the k8s lable size limit

* Update changelog for 5.1.0 release (#3676)

* Bump versions to 5.1.0 (#3677)

* fix port alignment for fog-ledger-router and grpc-gateway (#3687)

* Update changelog for 5.1.1 (#3688)

* Bump versions to 5.1.1 (#3689)

---------

Co-authored-by: Jason Greathouse <[email protected]>
Co-authored-by: Eran Rundstein <[email protected]>
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.

2 participants