Skip to content

Conversation

hajjimo
Copy link
Contributor

@hajjimo hajjimo commented Oct 3, 2025

Addresses #3453

Map Helm variable names to the updated Helm chart in https://github.com/interledger/charts/tree/main/charts/rafiki/values.yaml

Required

  • Used LinkOut component on external links
  • Reviewed Vale errors and made changes where appropriate
  • Ran Prettier
  • Previewed updates in Netlify
  • Received SME and/or peer approval if updates are significant
  • Included a "fixes #" comment

Conditional

  • Ensured sequence diagrams follow our style guide
  • Included code samples where appropriate
  • Updated related READMEs

Updated all Helm value names to use consolidated chart structure:
- Backend: config.backend.* prefix
- Auth: config.auth.* prefix
- Frontend: config.frontend.* prefix

Key changes:
- Fixed default values (USE_TIGERBEETLE=false, TRUST_PROXY=true)
- Added secret reference support for sensitive values
- Updated port numbers and configuration paths
- Added missing variables from codebase
- Corrected Helm value mappings to match actual chart
- Update of Helm variable names based on updated Helm chart
@github-actions github-actions bot added the pkg: documentation Changes in the documentation package. label Oct 3, 2025
Copy link

netlify bot commented Oct 3, 2025

Deploy Preview for brilliant-pasca-3e80ec ready!

Name Link
🔨 Latest commit 3391fdc
🔍 Latest deploy log https://app.netlify.com/projects/brilliant-pasca-3e80ec/deploys/68dfb764b1a84e0008e89600
😎 Deploy Preview https://deploy-preview-3679--brilliant-pasca-3e80ec.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@hajjimo hajjimo changed the title Mi/3453/helm env updates docs: 3453/helm env updates Oct 3, 2025
Copy link

github-actions bot commented Oct 3, 2025

🚀 Performance Test Results

Test Configuration:

  • VUs: 4
  • Duration: 1m0s

Test Metrics:

  • Requests/s: 38.26
  • Iterations/s: 12.78
  • Failed Requests: 0.00% (0 of 2302)
📜 Logs

> [email protected] run-tests:testenv /home/runner/work/rafiki/rafiki/test/performance
> ./scripts/run-tests.sh -e test "-k" "-q" "--vus" "4" "--duration" "1m"

Cloud Nine GraphQL API is up: http://localhost:3101/graphql
Cloud Nine Wallet Address is up: http://localhost:3100/
Happy Life Bank Address is up: http://localhost:4100/
cloud-nine-wallet-test-backend already set
cloud-nine-wallet-test-auth already set
happy-life-bank-test-backend already set
happy-life-bank-test-auth already set
     data_received..................: 831 kB 14 kB/s
     data_sent......................: 1.8 MB 29 kB/s
     http_req_blocked...............: avg=8.07µs   min=2.28µs   med=5.74µs  max=1.15ms   p(90)=7.02µs   p(95)=7.7µs   
     http_req_connecting............: avg=1.6µs    min=0s       med=0s      max=1.09ms   p(90)=0s       p(95)=0s      
     http_req_duration..............: avg=103.9ms  min=11.1ms   med=82.95ms max=579.14ms p(90)=183.48ms p(95)=206.54ms
       { expected_response:true }...: avg=103.9ms  min=11.1ms   med=82.95ms max=579.14ms p(90)=183.48ms p(95)=206.54ms
     http_req_failed................: 0.00%  ✓ 0         ✗ 2302
     http_req_receiving.............: avg=105.06µs min=28.08µs  med=90.06µs max=2.83ms   p(90)=132.43µs p(95)=166.02µs
     http_req_sending...............: avg=39.4µs   min=10.48µs  med=30.3µs  max=1.81ms   p(90)=43.67µs  p(95)=60.58µs 
     http_req_tls_handshaking.......: avg=0s       min=0s       med=0s      max=0s       p(90)=0s       p(95)=0s      
     http_req_waiting...............: avg=103.76ms min=10.84ms  med=82.83ms max=578.92ms p(90)=183.23ms p(95)=206.23ms
     http_reqs......................: 2302   38.262852/s
     iteration_duration.............: avg=312.73ms min=195.55ms med=301.7ms max=1.14s    p(90)=376.02ms p(95)=443.17ms
     iterations.....................: 769    12.781987/s
     vus............................: 4      min=4       max=4 
     vus_max........................: 4      min=4       max=4 

@melissahenderson
Copy link
Contributor

I can't speak to the veracity, but otherwise it looks good :)

I did want to call out that you changed some tables to list the variables in alphabetical order, while others stayed the way they are. You can leave them as is, but I wanted to point that out in case alphabetizing all of them was your intent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: documentation Changes in the documentation package. user-docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants