-
Notifications
You must be signed in to change notification settings - Fork 339
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: support environment variable in config file #713
Conversation
Codecov Report
@@ Coverage Diff @@
## master #713 +/- ##
==========================================
+ Coverage 32.36% 32.45% +0.08%
==========================================
Files 65 64 -1
Lines 6488 6499 +11
==========================================
+ Hits 2100 2109 +9
- Misses 4137 4138 +1
- Partials 251 252 +1
Continue to review full report at Codecov.
|
can you add an e2e test case to cover this? |
OK |
Signed-off-by: Jintao Zhang <zhangjintao9020@gmail.com>
Please answer these questions before submitting a pull request
Why submit this pull request?
Bugfix
New feature provided
Improve performance
Backport patches
Related issues
request help: Apisix ingress controller config.yaml couldn't read os environment variable #710