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

watches was loaded many times #4093

Closed
OPSTime opened this issue May 7, 2018 · 2 comments
Closed

watches was loaded many times #4093

OPSTime opened this issue May 7, 2018 · 2 comments
Labels
waiting-reply Waiting on response from Original Poster or another individual in the thread

Comments

@OPSTime
Copy link

OPSTime commented May 7, 2018

watches configuration:

"watches": [
{
"type": "key",
"key": "bench",
"token": "d30f6835-535c-40ab-bce9-e14e8f4a8077",
"handler_type": "script",
"args": ["/data/shell/t.sh"]
}
]

/data/shell/t.sh

#!/bin/bash
echo 1 >>/tmp/out.log

consul.service

[Unit]
Description=Consul service discovery agent
Requires=network-online.target
After=network-online.target

[Service]
Type=simple
User=consul
Group=consul
Restart=on-failure
Environment=GOMAXPROCS=8
ExecStartPre=/usr/local/consul/bin/consul validate -quiet /usr/local/consul/etc/
ExecStart=/usr/local/consul/bin/consul agent -config-dir=/usr/local/consul/etc/
ExecReload=/bin/kill -HUP $MAINPID
KillSignal=SIGTERM
TimeoutStopSec=5

[Install]
WantedBy=multi-user.target

run command

systemctl reload consul
systemctl reload consul

change key: bench value

cat /tmp/out.log

1
1

@pearkes
Copy link
Contributor

pearkes commented May 7, 2018

I believe you are asking about the behavior of watches during reload -- are you seeing the issue described here: #571?

@pearkes pearkes added the waiting-reply Waiting on response from Original Poster or another individual in the thread label May 7, 2018
@pearkes
Copy link
Contributor

pearkes commented May 23, 2018

Given we haven't heard anything based on our suggestions/questions above I'm going to close this issue, but I encourage you to comment and we can re-open it if you want to pick this up again. I still believe this is a duplicate of #571.

Alternatively, if things have changed dramatically, feel free to create a new issue or PR.

@pearkes pearkes closed this as completed May 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-reply Waiting on response from Original Poster or another individual in the thread
Projects
None yet
Development

No branches or pull requests

2 participants