Closed harudark closed 4 years ago
@harudark Should we close MRs https://github.com/bosun-monitor/bosun/pull/2426, https://github.com/bosun-monitor/bosun/pull/2427 and https://github.com/bosun-monitor/bosun/pull/2442 then?
@harudark Should we close MRs #2426, #2427 and #2442 then?
I'll fix issues mentioned in code review and then close those MRs.
any news on that?
@svagner @harudark Have you had a chance to make the last few changes to this ?
Description
This pull request adds ES 7.x support in Bosun. I submitted https://github.com/bosun-monitor/bosun/pull/2427 before but that is based on my own go modules migration pull request which is different from what upstream maintainers did. This one is based on upstream go modules migration.
Type of change
From the following, please check the options that are relevant.
How has this been tested?
Tested with ES 7.2 and also ES 6.8 to make sure nothing breaks.
Checklist: