fix: haproxy fails to start by jeromy-cannon · Pull Request #921 · hashgraph/full-stack-testing · GitHub
Skip to content

fix: haproxy fails to start#921

Merged
jeromy-cannon merged 3 commits into
mainfrom
00920-haproxy-bug-fix
Jul 16, 2024
Merged

fix: haproxy fails to start#921
jeromy-cannon merged 3 commits into
mainfrom
00920-haproxy-bug-fix

Conversation

@jeromy-cannon

@jeromy-cannon jeromy-cannon commented Jul 15, 2024

Copy link
Copy Markdown
Contributor

Description

This pull request changes the following:

  • helm dependency update chart.lock
  • updated haproxy.cfg to set password via environment variable from secret for dataplane user and stats user & password
  • updated haproxy liveness/readiness/startup probe to work with password coming from environment variable

Related Issues

Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
@github-actions

Copy link
Copy Markdown

@codecov

codecov Bot commented Jul 15, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.17%. Comparing base (dbed2f9) to head (cc87ce1).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #921   +/-   ##
=========================================
  Coverage     67.17%   67.17%           
  Complexity      339      339           
=========================================
  Files            64       64           
  Lines          1319     1319           
  Branches        157      157           
=========================================
  Hits            886      886           
  Misses          370      370           
  Partials         63       63           

Impacted file tree graph

@sonarqubecloud

Copy link
Copy Markdown

Comment thread charts/fullstack-deployment/config-files/haproxy.cfg
@jeromy-cannon jeromy-cannon merged commit e162d3d into main Jul 16, 2024
@jeromy-cannon jeromy-cannon deleted the 00920-haproxy-bug-fix branch July 16, 2024 10:19
swirlds-automation added a commit that referenced this pull request Aug 1, 2024
## [0.29.0](v0.28.3...v0.29.0) (2024-08-01)

### Features

* add extra container for sidecar uploader ([#926](#926)) ([58ff515](58ff515))

### Bug Fixes

* haproxy fails to start ([#921](#921)) ([e162d3d](e162d3d))
* incorrect NODE_NAMES ([#925](#925)) ([81629a8](81629a8))
@swirlds-automation

Copy link
Copy Markdown
Contributor

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[Network]: BUG: haproxy fails to start Charts[Network]: haproxy.cfg stats auth is using random which changes upon helm upgrade

4 participants