-
Notifications
You must be signed in to change notification settings - Fork 190
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
Opsgenie Integration requirements #239
Comments
Hi @lumi-yuri Thanks so much for doing the groundwork on this. My only concern is that some of the default check scripts might be still using python2. Have you noticed any issues with your build? |
@jjethwa thank you for your quick response on this. I did not notice any issue yet and currently working on cutting over from our old Icinga1 to new Icinga2 running inside the container created from your Image and integrated with OpsGenie and slack. Everything works as charm so far with a bit of crafty modification to the image that I mentioned: default python and python3-request apt package install. However, I am not using any of existing default nagios plugins(we are using Icinga2 --> nagios integration) that are written in python. At the same time python 2 reached its EOL seems like on January 1st 2020. I am also happy to share my OpsGenie OEC connector integration config files if that can be helpful for someone here.
Thank you in advance, Yuri. |
Thanks @lumi-yuri I'll do some testing over the next few days with the default container setup and python3 and if I don't run into any issues, add the lines as you suggested. Your config files would be really helpful and thanks for wanting to share them. What I would suggest is creating a public git repository to share them, similar to what I did for the Slack notification script and config files: https://github.com/jjethwa/icinga2-slack-notification There's also the Icinga Exchange: https://exchange.icinga.com/ |
@jjethwa thank you very much. There will be two extra lines as a desired state. One will make python3 as default and the other one will install `# Starting OpsGenie Connector if it is not running
Thank you again in advance, Yuri. |
@lumi-yuri How about using Supervisor to manage the connector? It's already being used in the container to run the other daemons like Icinga2, MariaDB, httpd, etc |
@jjethwa it is not a bad idea, but we are trying to avoid modifying your base image and building our own from yours. That will make our lives easier with upgrades and such. At some point I was thinking of going this path but then if we only have to execute one command: Best, Yuri. |
Hi @lumi-yuri Sorry for the delay, I'm still working on this, but a bit slower as it's crunch time at work. I should be able to get it done late next week. |
No worries @jjethwa. Take your time. |
Hi there, I have been recently working on integrating new OpsGenie OEC connector with Icinga2 container and faced couple of things that I have decided to ask you about. OEC connector is using python3 scripts to communicate with Icinga2 and has one python module requirement(python3-requests). Is it possible and of course if you feel ok making two small adjustments to Icinga2 image:
cd /usr/bin && ln -sf python3 python
python3-requests
apt package inside the Image.Currently, I made it working in jenkins pipeline using the following adjustments, but they can be simplified to just only starting OEC connector which would be great.
Thank you very much in advance for all the comments and replies. Hope this request will reach the right place.
Best, Yuri.
The text was updated successfully, but these errors were encountered: