Closed okdas closed 1 year ago
Patch coverage: 100.00
% and project coverage change: -0.04
:warning:
Comparison is base (
f72e1f0
) 31.38% compared to head (2c28cc3
) 31.35%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
@okdas I'm just now noticing this, I've been working in the same area, possibly in a slightly different direction. I will take a comprehensive look at this when I have a minute. In the meantime, I wanted to call out #806.
@bryanchriswhite oh, seems like your PR would resolve the issue I've been trying to address here! :) Let's focus on your PR then, since it went through some reviews already.
Description
This PR adds a
POCKET_RPC_URL
environment variable. We are going to use it as a fallback in the CLI if theremote_cli_url
flag is not specified. Similarly, that environment variable also has a fallback default value, which ishttp://localhost:50832
— the same default value we've always used.In addition, I've replaced an existing variable,
RPC_HOST
, which was previously used in the cluster manager and e2e tests, with this new variable to maintain consistency.Once this is merged, I'll update the DevNet infrastructure, so it will also use the new configuration.
Summary generated by Reviewpad on 08 Jun 23 23:29 UTC
This pull request adds support for a POCKET_RPC_URL environment variable, replacing the older RPC_HOST variable used in several places across the codebase. It also updates two URLs in the docker-compose and CLI client manifests. Finally, it removes two unused variables and a redundant import.
Issue
Fixes #814
Type of change
Please mark the relevant option(s):
List of changes
POCKET_RPC_URL
RPC_HOST
env variable withPOCKET_RPC_URL
Testing
make develop_test
; if any code changes were mademake test_e2e
on k8s LocalNet; if any code changes were madee2e-devnet-test
passes tests on DevNet; if any code was changedRequired Checklist
godoc
format comments on touched members (see: tip.golang.org/doc/comment)If Applicable Checklist
shared/docs/*
if I updatedshared/*
README(s)