sebastien-coavoux
sebastien-coavoux
Hey, I've just pushed a commit to start work on this item. Very basic, no test added whatsoever. You can have a first look and let me know . It...
Hey, any update on it? I gpg signed the commit but now the generator fails because it is not finding some OID (raritan). Pretty strange since I have touched the...
Hi, I've rebased the MR on main so that the CI is green. I've also changed the vocab from instance to index/indices ( I had to choose a side for...
Done !
@SuperQ sorry for the delay, I was mostly off in summer. I've just added the index validation as part of Unmarshalling.
@PrestonTaylor , is there anything not clear in https://github.com/prometheus/snmp_exporter/pull/624/files#diff-a715bdcf88cd771d7ae09bd01863bab314ff194ba28e984689e9040d87846db9R147 ? Readme in generator provides an example, maybe you meant the Readme at the top level of the repo ?
@PrestonTaylor Ok I got it. I have added the 2 oid, the example provided is accurate with it. Thanks !
@SuperQ in case you missed it :point_up:
Updated PR from the input. Also rebased on main
I haven't dive into it but it looks like that the panic is not coming from the code modified by the PR. `Mar 20 18:56:07 myexporterserver snmp_exporter: panic: Unknown index...