Skip to content

Conversation

@njlie
Copy link
Contributor

@njlie njlie commented Aug 14, 2025

Changes proposed in this pull request

  • Fixes a typo in the default environment config for backend.
  • Fixes a pre-request-script when resolving wallet addresses in the Bruno examples.
  • Fixes an issue where a tenanted peer would not be seeded if it was specified in the seed file.

Context

Fixes RAF-1143.

Checklist

  • Related issues linked using fixes #number
  • Tests added/updated
  • Make sure that all checks pass
  • Bruno collection updated (if necessary)
  • Documentation issue created with user-docs label (if necessary)
  • OpenAPI specs updated (if necessary)

@netlify
Copy link

netlify bot commented Aug 14, 2025

Deploy Preview for brilliant-pasca-3e80ec canceled.

Name Link
🔨 Latest commit 352d28a
🔍 Latest deploy log https://app.netlify.com/projects/brilliant-pasca-3e80ec/deploys/689e5ab4a0d71700082dfb3f

@github-actions
Copy link

🚀 Performance Test Results

Test Configuration:

  • VUs: 4
  • Duration: 1m0s

Test Metrics:

  • Requests/s: 42.27
  • Iterations/s: 14.11
  • Failed Requests: 0.00% (0 of 2541)
📜 Logs

> performance@1.0.0 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..................: 917 kB 15 kB/s
     data_sent......................: 2.0 MB 33 kB/s
     http_req_blocked...............: avg=6.57µs  min=2.53µs   med=5.05µs   max=849.55µs p(90)=6.09µs   p(95)=6.58µs  
     http_req_connecting............: avg=323ns   min=0s       med=0s       max=230.03µs p(90)=0s       p(95)=0s      
     http_req_duration..............: avg=93.98ms min=7.08ms   med=80.76ms  max=654.64ms p(90)=159.2ms  p(95)=182.14ms
       { expected_response:true }...: avg=93.98ms min=7.08ms   med=80.76ms  max=654.64ms p(90)=159.2ms  p(95)=182.14ms
     http_req_failed................: 0.00%  ✓ 0         ✗ 2541
     http_req_receiving.............: avg=84.89µs min=23.75µs  med=72.99µs  max=2.68ms   p(90)=109.13µs p(95)=139.16µs
     http_req_sending...............: avg=35.28µs min=9.03µs   med=27.23µs  max=1.38ms   p(90)=39.76µs  p(95)=53.96µs 
     http_req_tls_handshaking.......: avg=0s      min=0s       med=0s       max=0s       p(90)=0s       p(95)=0s      
     http_req_waiting...............: avg=93.86ms min=6.91ms   med=80.55ms  max=654.56ms p(90)=159.09ms p(95)=182.04ms
     http_reqs......................: 2541   42.274164/s
     iteration_duration.............: avg=283.2ms min=184.75ms med=270.32ms max=1.1s     p(90)=335.3ms  p(95)=368.3ms 
     iterations.....................: 848    14.108025/s
     vus............................: 4      min=4       max=4 
     vus_max........................: 4      min=4       max=4 

Copy link
Contributor

@BlairCurrey BlairCurrey 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. tested against the open payments example (looks like create quote failing with 403 on main)

@njlie njlie merged commit 8e3cf96 into main Aug 15, 2025
44 checks passed
@njlie njlie deleted the nl/raf-1143/fix-local-env branch August 15, 2025 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants