You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by GitBox <gi...@apache.org> on 2020/03/10 11:23:41 UTC
[GitHub] [skywalking] kezhenxu94 opened a new pull request #4483: [BugFix]
Docker InfluxDB cannot be enable
kezhenxu94 opened a new pull request #4483: [BugFix] Docker InfluxDB cannot be enable
URL: https://github.com/apache/skywalking/pull/4483
### Motivation:
This patch fixes the bug that there's no chance to enable the InfluxDB storage in docker container.
### Modifications:
Generate InfluxDB configuration section to enable it.
### Result:
Users can enable InfluxDB by specifying SW_STORAGE=influxdb
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [skywalking] wu-sheng commented on issue #4483: [BugFix] Docker
InfluxDB cannot be enabled
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #4483: [BugFix] Docker InfluxDB cannot be enabled
URL: https://github.com/apache/skywalking/pull/4483#issuecomment-597097770
Why you merge this PR as CI breaks?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [skywalking] codecov-io commented on issue #4483: [BugFix] Docker
InfluxDB cannot be enabled
Posted by GitBox <gi...@apache.org>.
codecov-io commented on issue #4483: [BugFix] Docker InfluxDB cannot be enabled
URL: https://github.com/apache/skywalking/pull/4483#issuecomment-597050063
# [Codecov](https://codecov.io/gh/apache/skywalking/pull/4483?src=pr&el=h1) Report
> Merging [#4483](https://codecov.io/gh/apache/skywalking/pull/4483?src=pr&el=desc) into [master](https://codecov.io/gh/apache/skywalking/commit/ab8d7fab23b69593d7d06bdee20421006cb20189?src=pr&el=desc) will **decrease** coverage by `0.02%`.
> The diff coverage is `n/a`.
[![Impacted file tree graph](https://codecov.io/gh/apache/skywalking/pull/4483/graphs/tree.svg?width=650&token=qrILxY5yA8&height=150&src=pr)](https://codecov.io/gh/apache/skywalking/pull/4483?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #4483 +/- ##
==========================================
- Coverage 24.97% 24.95% -0.03%
==========================================
Files 1231 1231
Lines 28496 28496
Branches 3905 3905
==========================================
- Hits 7118 7110 -8
- Misses 20719 20726 +7
- Partials 659 660 +1
```
| [Impacted Files](https://codecov.io/gh/apache/skywalking/pull/4483?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...walking/oap/server/core/analysis/Downsampling.java](https://codecov.io/gh/apache/skywalking/pull/4483/diff?src=pr&el=tree#diff-b2FwLXNlcnZlci9zZXJ2ZXItY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2t5d2Fsa2luZy9vYXAvc2VydmVyL2NvcmUvYW5hbHlzaXMvRG93bnNhbXBsaW5nLmphdmE=) | `0% <0%> (-100%)` | :arrow_down: |
| [...ache/skywalking/apm/agent/core/jvm/JVMService.java](https://codecov.io/gh/apache/skywalking/pull/4483/diff?src=pr&el=tree#diff-YXBtLXNuaWZmZXIvYXBtLWFnZW50LWNvcmUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NreXdhbGtpbmcvYXBtL2FnZW50L2NvcmUvanZtL0pWTVNlcnZpY2UuamF2YQ==) | `55% <0%> (-1.67%)` | :arrow_down: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/skywalking/pull/4483?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/skywalking/pull/4483?src=pr&el=footer). Last update [ab8d7fa...d49310b](https://codecov.io/gh/apache/skywalking/pull/4483?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [skywalking] kezhenxu94 merged pull request #4483: [BugFix] Docker
InfluxDB cannot be enabled
Posted by GitBox <gi...@apache.org>.
kezhenxu94 merged pull request #4483: [BugFix] Docker InfluxDB cannot be enabled
URL: https://github.com/apache/skywalking/pull/4483
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [skywalking] dmsolr removed a comment on issue #4483: [BugFix]
Docker InfluxDB cannot be enabled
Posted by GitBox <gi...@apache.org>.
dmsolr removed a comment on issue #4483: [BugFix] Docker InfluxDB cannot be enabled
URL: https://github.com/apache/skywalking/pull/4483#issuecomment-597057939
beginning to refactor e2e framework?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [skywalking] dmsolr commented on issue #4483: [BugFix] Docker
InfluxDB cannot be enabled
Posted by GitBox <gi...@apache.org>.
dmsolr commented on issue #4483: [BugFix] Docker InfluxDB cannot be enabled
URL: https://github.com/apache/skywalking/pull/4483#issuecomment-597057939
beginning to refactor e2e framework?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services