Bug#921739: amtool: error: comment required by config
On Thu, 14 Feb 2019 10:10:47 +0100 Santiago Vila wrote: > Ok, that worked, but I think it would be much better if such option > could be added to the README.md.gz somewhere, or a README.Debian, or > something. That will avoid searches in search engines, stackoverflow > or the Debian BTS. > > There is yet another thing I didn't find anywhere and I would like to > see documented somewhere in /usr/share/doc/prometheus-alertmanager: > > How do I add a silence which expires after one hour, for example? > > (The web UI allowed this, and in some sense it's basic stuff, so I > think it should be documented as well). > > (Can you tell upstream to extend the README.md a little bit?) > > Thanks. There are already quite a few examples on how to use amtool in the upstream README.md. I don't think that README.Debian is the right place to add additional examples that are not specific to running amtool on Debian. Might I suggest that if you still find the upstream documentation lacking, you open a pull request upstream? The Prometheus documentation is open source, and they welcome contributions.
Bug#921739: amtool: error: comment required by config
On Sat, Feb 09, 2019 at 12:06:50PM +, Martín Ferrari wrote: > Hi Santiago, > > On 08/02/2019 17:35, Santiago Vila wrote: > > > File /usr/share/doc/prometheus-alertmanager/README.md.gz suggests doing > > this: > > > > amtool silence add alertname=Test_Alert > > > > but this is what I get: > > > > amtool: error: comment required by config > > I don't know how that is configured, but you just need to add a comment > to the silence like '--comment="testing silences"' Ok, that worked, but I think it would be much better if such option could be added to the README.md.gz somewhere, or a README.Debian, or something. That will avoid searches in search engines, stackoverflow or the Debian BTS. There is yet another thing I didn't find anywhere and I would like to see documented somewhere in /usr/share/doc/prometheus-alertmanager: How do I add a silence which expires after one hour, for example? (The web UI allowed this, and in some sense it's basic stuff, so I think it should be documented as well). (Can you tell upstream to extend the README.md a little bit?) Thanks.
Bug#921739: amtool: error: comment required by config
Hi Santiago, On 08/02/2019 17:35, Santiago Vila wrote: > File /usr/share/doc/prometheus-alertmanager/README.md.gz suggests doing this: > > amtool silence add alertname=Test_Alert > > but this is what I get: > > amtool: error: comment required by config I don't know how that is configured, but you just need to add a comment to the silence like '--comment="testing silences"' > Trying to see what's wrong, I tried using amtool 0.16.1, downloaded > from github: > > $ /tmp/amtool silence add alertname=Test_Alert > amtool: error: required flag --alertmanager.url not provided > > Ok, apparently it needs the url of the current alertmanager instance, > let's try this then: Yes, the debian binary comes with a sane default which is not present in upstream. > I would love to use the web UI, but it's gone: > > The Debian package of the alertmanager does not include a web > application. > > Is this a Debian-specific change? If yes, could we have the web UI back? > In my opinion, it was better than nothing. The problem is that upstream switched to Elm for generating the UI, and that would have required preparing and uploading dozens of new packages, and I did not have the resources for that. Somebody offered to write a script to download the pre-compiled UI and install it locally, but never happened. -- Martín Ferrari (Tincho)
Bug#921739: amtool: error: comment required by config
Package: prometheus-alertmanager Version: 0.15.3+ds-2 Hello again. File /usr/share/doc/prometheus-alertmanager/README.md.gz suggests doing this: amtool silence add alertname=Test_Alert but this is what I get: amtool: error: comment required by config Trying to see what's wrong, I tried using amtool 0.16.1, downloaded from github: $ /tmp/amtool silence add alertname=Test_Alert amtool: error: required flag --alertmanager.url not provided Ok, apparently it needs the url of the current alertmanager instance, let's try this then: $ /tmp/amtool --alertmanager.url=http//:localhost:9093 silence add alertname=Test_Alert Output: amtool: error: comment required by config So, back to square one. Is there a trick to make this work? Maybe I'm missing something, but I don't know what it could be. I would love to use the web UI, but it's gone: The Debian package of the alertmanager does not include a web application. Is this a Debian-specific change? If yes, could we have the web UI back? In my opinion, it was better than nothing. Thanks.
