Skip to content
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

types.db: fix ping_droprate range spec #3244

Merged
merged 1 commit into from Feb 25, 2020
Merged

Conversation

yousong
Copy link
Contributor

@yousong yousong commented Aug 8, 2019

Changelog: types.db: fix ping_droprate range spec

It's always in range 0-1 since the initial support for the data source
in commit 9cc8e54

yousong added a commit to yousong/packages that referenced this pull request Aug 8, 2019
Upstream project's behaviour of reporting this data source in range 0-1
is used by at least CentOS 7 collectd 5.8.1, very likely the same
behaviour is also in widely use by other distributions.

The change is important for providing consistent experience across
collectd deployments on different environments

Ref: collectd/collectd#3244
Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
yousong added a commit to yousong/packages that referenced this pull request Aug 8, 2019
Upstream project's behaviour of reporting this data source in range 0-1
is used by at least CentOS 7 collectd 5.8.1, very likely the same
behaviour is also in widely use by other distributions.

The change is important for providing consistent experience across
collectd deployments on different environments

Ref: collectd/collectd#3244
Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
@yousong
Copy link
Contributor Author

yousong commented Aug 9, 2019

Not likely the ci check failure was caused by the change here.

  CCLD     write_log.la
cd buildperl && /Applications/Xcode-10.1.app/Contents/Developer/usr/bin/make
Makefile out-of-date with respect to Makefile.PL
Cleaning current config before rebuilding Makefile...
make -f Makefile.old clean > /dev/null 2>&1
/usr/bin/perl Makefile.PL "PREFIX=/Users/travis/build/collectd/collectd/collectd-5.8.1.git/_inst"
Writing Makefile for Collectd
Writing MYMETA.yml and MYMETA.json
==> Your Makefile has been rebuilt. <==
==> Please rerun the make command.  <==
false
make[3]: *** [Makefile] Error 1
make[2]: *** [perl] Error 2
make[2]: *** Waiting for unfinished jobs....
  CCLD     collectd-nagios
make[1]: *** [all] Error 2
make: *** [distcheck] Error 1
The command "make distcheck DISTCHECK_CONFIGURE_FLAGS="--disable-dependency-tracking --enable-debug"" exited with 2.

yousong added a commit to yousong/packages that referenced this pull request Aug 9, 2019
Upstream project's behaviour of reporting this data source in range 0-1
is used by at least CentOS 7 collectd 5.8.1, very likely the same
behaviour is also in widely use by other distributions.

The change is important for providing consistent experience across
collectd deployments on different environments

Ref: collectd/collectd#3244
Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
yousong added a commit to openwrt/packages that referenced this pull request Aug 9, 2019
Upstream project's behaviour of reporting this data source in range 0-1
is used by at least CentOS 7 collectd 5.8.1, very likely the same
behaviour is also in widely use by other distributions.

The change is important for providing consistent experience across
collectd deployments on different environments

Ref: collectd/collectd#3244
Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
yousong added a commit to openwrt/packages that referenced this pull request Aug 10, 2019
Upstream project's behaviour of reporting this data source in range 0-1
is used by at least CentOS 7 collectd 5.8.1, very likely the same
behaviour is also in widely use by other distributions.

The change is important for providing consistent experience across
collectd deployments on different environments

Ref: collectd/collectd#3244
Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
It's always in range 0-1 since the initial support for the data source
in commit 9cc8e54
aparcar pushed a commit to aparcar/packages that referenced this pull request Aug 27, 2019
Upstream project's behaviour of reporting this data source in range 0-1
is used by at least CentOS 7 collectd 5.8.1, very likely the same
behaviour is also in widely use by other distributions.

The change is important for providing consistent experience across
collectd deployments on different environments

Ref: collectd/collectd#3244
Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
@yousong
Copy link
Contributor Author

yousong commented Oct 15, 2019

Friendly ping while I am working on collectd again.

yousong added a commit to openwrt/packages that referenced this pull request Nov 14, 2019
Upstream project's behaviour of reporting this data source in range 0-1
is used by at least CentOS 7 collectd 5.8.1, very likely the same
behaviour is also in widely use by other distributions.

The change is important for providing consistent experience across
collectd deployments on different environments

Ref: collectd/collectd#3244
Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
(cherry picked from commit 2f66fa5)
Ref: openwrt/luci#3308
@yousong
Copy link
Contributor Author

yousong commented Nov 15, 2019

Ping @octo

Copy link
Contributor

@dago dago left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dago dago merged commit 17699d8 into collectd:master Feb 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants