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

feat: use production configuration for envoy and haproxy configuration #154

Merged
merged 2 commits into from
Jul 10, 2023

Conversation

deepak-swirlds
Copy link
Contributor

Description

This pull request changes the following:

Uses production configuration for

  • envoy proxy
  • haproxy

Note:
current the containers are failing to start due to the configuration, this will be fixed before merging

Related Issues

Signed-off-by: Deepak Mishra <deepak@swirldslabs.com>
@deepak-swirlds deepak-swirlds added this to the v0.3.0 milestone Jul 10, 2023
@deepak-swirlds deepak-swirlds self-assigned this Jul 10, 2023
@github-actions
Copy link

github-actions bot commented Jul 10, 2023

Unit Test Results

38 tests  ±0   35 ✔️ ±0   29s ⏱️ ±0s
  9 suites ±0     3 💤 ±0 
  9 files   ±0     0 ±0 

Results for commit cd5d8b1. ± Comparison against base commit 266a0b0.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Jul 10, 2023

Codecov Report

Merging #154 (cd5d8b1) into main (266a0b0) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #154   +/-   ##
=========================================
  Coverage     56.70%   56.70%           
  Complexity      144      144           
=========================================
  Files            36       36           
  Lines           723      723           
  Branches         73       73           
=========================================
  Hits            410      410           
  Misses          281      281           
  Partials         32       32           

Impacted file tree graph

@nathanklick nathanklick changed the title using production configuration for envoy and haproxy configuration feat: use production configuration for envoy and haproxy configuration Jul 10, 2023
Signed-off-by: Nathan Klick <nathan@swirldslabs.com>

# Conflicts:
#	charts/hedera-network/templates/configmaps.yaml
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@nathanklick nathanklick merged commit 6f45361 into main Jul 10, 2023
@nathanklick nathanklick deleted the bugfix/153-haproxy-envoyproxy-production-config branch July 10, 2023 23:32
swirlds-automation added a commit that referenced this pull request Jul 18, 2023
## [0.3.0](v0.2.0...v0.3.0) (2023-07-18)

### Features

* add unit tests to validate logging behaviors in Helm Client ([#151](#151)) ([5a826d2](5a826d2))
* implement backup uploader sidecar container ([#148](#148)) ([7854131](7854131))
* implement helm chart test ([#152](#152)) ([9321e4c](9321e4c))
* run code analysis as a separate privileged job ([#171](#171)) ([1da604d](1da604d))
* use production configuration for envoy and haproxy configuration ([#154](#154)) ([6f45361](6f45361))

### Bug Fixes

* Helm chart statefulset has incorrect config for uploader sidecars  ([#150](#150)) ([eb52514](eb52514))
* root container should run in privilege mode as root user ([#146](#146)) ([f6d372b](f6d372b))
@swirlds-automation
Copy link
Contributor

🎉 This PR is included in version 0.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Charts[Proxy]: Haproxy and envoy proxy should have production configuration
3 participants