Re: [prometheus-developers] Remove /api/v2 of Prometheus
Let's add note as well to notify people! (That we mark v2 as deprecated) :p On Fri, 28 Aug 2020, 15:14 Bartłomiej Płotka, wrote: > Let's remove +1 > > On Fri, 28 Aug 2020, 14:41 Simon Pasquier, wrote: > >> On Thu, Aug 27, 2020 at 4:40 PM Julien Pivotto >> wrote: >> > >> > On 27 Aug 16:30, Bjoern Rabenstein wrote: >> > > On 24.08.20 09:35, Brian Brazil wrote: >> > > > On Sun, 23 Aug 2020 at 16:56, Julien Pivotto < >> roidelapl...@prometheus.io> >> > > > wrote: >> > > > >> > > > On 23 Aug 12:17, Brian Brazil wrote: >> > > > > On Sun, 23 Aug 2020 at 12:08, Julien Pivotto < >> roidelapl...@prometheus.io> >> > > > > wrote: >> > > > > >> > > > > > Dear developers, >> > > > > > >> > > > > > Prometheus has an experimental api V2 in GRPC, which does >> not get any >> > > > > > updates. In addition, it is marked experimental, so we >> could drop it in >> > > > > > minor releases. >> > > > > > >> > > > > > The code itself is quite small, there is (to my knowledge) >> no >> > > > > > documentation and no tests. >> > > > > > >> > > > > > The feature surface seems to be really small, with only >> admin-specific >> > > > > > API's implemented. >> > > > > > >> > > > > > At this point, I'd like to see if we could remove that >> code, and also >> > > > > > the cmux library we use, which did not get any updated in 2 >> years, and >> > > > > > is incompatible with recent releases of GRPC. >> > > > > >> > > > > >> > > > > The plan of record is to remove it whenever 3.0 is released, >> however if >> > > > > it's causing us hassle I see no issue with removing it sooner. >> > > > >> > > > 1. CMUX >> > > > >> > > > That cmux library is using the helloworld package from grpc >> which is >> > > > gone. >> > > > https://github.com/soheilhy/cmux/blob/ >> > > > 8a8ea3c53959009183d7914522833c1ed8835020/example_test.go#L32 >> > > > >> > > > Additionally, the 2.21 fixes some misconfiguration of the cmux >> that >> > > > makes prometheus hang on SIGTERM. I hope that the mitigation we >> have is >> > > > complete, but the library lacks features like clean shutdown: >> > > > https://github.com/soheilhy/cmux/issues/39 >> > > > >> > > > At the end the repo seems abandoned since 2 years. >> > > > >> > > > 2. GRPC >> > > > >> > > > We can not update GRPC, partly because of cmux, partly because >> we are >> > > > friendly with downstream, cortex, which depends on etcd, which >> > > > requires an old version of grpc. >> > > > >> > > > This is blocking an update of google.golang.org/api, which is >> used >> > > > in our GCE discovery. >> > > > >> > > > >> > > > Those sound like good enough reasons to me. >> > > >> > > +1 >> > >> > Do we want to add in the 2.21 release notes that we will remove it in >> > 2.22? >> >> Sounds good to me. >> >> > >> > -- >> > Julien Pivotto >> > @roidelapluie >> > >> > -- >> > You received this message because you are subscribed to the Google >> Groups "Prometheus Developers" group. >> > To unsubscribe from this group and stop receiving emails from it, send >> an email to prometheus-developers+unsubscr...@googlegroups.com. >> > To view this discussion on the web visit >> https://groups.google.com/d/msgid/prometheus-developers/20200827144020.GA601189%40oxygen >> . >> > >> >> -- >> You received this message because you are subscribed to the Google Groups >> "Prometheus Developers" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to prometheus-developers+unsubscr...@googlegroups.com. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/prometheus-developers/CAM6RFu7R_JWv4b11QnEB1AwhFKFTZgWKBG6N6iHJ6arEMbe4cA%40mail.gmail.com >> . >> > -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/CAMssQwZjxisVTVdEk%3D2WvVYks%2Bme4iMA3ro7bR_Bf8rXnn0pXQ%40mail.gmail.com.
Re: [prometheus-developers] Remove /api/v2 of Prometheus
Let's remove +1 On Fri, 28 Aug 2020, 14:41 Simon Pasquier, wrote: > On Thu, Aug 27, 2020 at 4:40 PM Julien Pivotto > wrote: > > > > On 27 Aug 16:30, Bjoern Rabenstein wrote: > > > On 24.08.20 09:35, Brian Brazil wrote: > > > > On Sun, 23 Aug 2020 at 16:56, Julien Pivotto < > roidelapl...@prometheus.io> > > > > wrote: > > > > > > > > On 23 Aug 12:17, Brian Brazil wrote: > > > > > On Sun, 23 Aug 2020 at 12:08, Julien Pivotto < > roidelapl...@prometheus.io> > > > > > wrote: > > > > > > > > > > > Dear developers, > > > > > > > > > > > > Prometheus has an experimental api V2 in GRPC, which does > not get any > > > > > > updates. In addition, it is marked experimental, so we could > drop it in > > > > > > minor releases. > > > > > > > > > > > > The code itself is quite small, there is (to my knowledge) no > > > > > > documentation and no tests. > > > > > > > > > > > > The feature surface seems to be really small, with only > admin-specific > > > > > > API's implemented. > > > > > > > > > > > > At this point, I'd like to see if we could remove that code, > and also > > > > > > the cmux library we use, which did not get any updated in 2 > years, and > > > > > > is incompatible with recent releases of GRPC. > > > > > > > > > > > > > > > The plan of record is to remove it whenever 3.0 is released, > however if > > > > > it's causing us hassle I see no issue with removing it sooner. > > > > > > > > 1. CMUX > > > > > > > > That cmux library is using the helloworld package from grpc > which is > > > > gone. > > > > https://github.com/soheilhy/cmux/blob/ > > > > 8a8ea3c53959009183d7914522833c1ed8835020/example_test.go#L32 > > > > > > > > Additionally, the 2.21 fixes some misconfiguration of the cmux > that > > > > makes prometheus hang on SIGTERM. I hope that the mitigation we > have is > > > > complete, but the library lacks features like clean shutdown: > > > > https://github.com/soheilhy/cmux/issues/39 > > > > > > > > At the end the repo seems abandoned since 2 years. > > > > > > > > 2. GRPC > > > > > > > > We can not update GRPC, partly because of cmux, partly because > we are > > > > friendly with downstream, cortex, which depends on etcd, which > > > > requires an old version of grpc. > > > > > > > > This is blocking an update of google.golang.org/api, which is > used > > > > in our GCE discovery. > > > > > > > > > > > > Those sound like good enough reasons to me. > > > > > > +1 > > > > Do we want to add in the 2.21 release notes that we will remove it in > > 2.22? > > Sounds good to me. > > > > > -- > > Julien Pivotto > > @roidelapluie > > > > -- > > You received this message because you are subscribed to the Google > Groups "Prometheus Developers" group. > > To unsubscribe from this group and stop receiving emails from it, send > an email to prometheus-developers+unsubscr...@googlegroups.com. > > To view this discussion on the web visit > https://groups.google.com/d/msgid/prometheus-developers/20200827144020.GA601189%40oxygen > . > > > > -- > You received this message because you are subscribed to the Google Groups > "Prometheus Developers" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to prometheus-developers+unsubscr...@googlegroups.com. > To view this discussion on the web visit > https://groups.google.com/d/msgid/prometheus-developers/CAM6RFu7R_JWv4b11QnEB1AwhFKFTZgWKBG6N6iHJ6arEMbe4cA%40mail.gmail.com > . > -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/CAMssQwZft7919_x6b40yFzwKa8g8QT9VNxfW%3DWFGZn6MxuhGSA%40mail.gmail.com.
Re: [prometheus-developers] Remove /api/v2 of Prometheus
On Thu, Aug 27, 2020 at 4:40 PM Julien Pivotto wrote: > > On 27 Aug 16:30, Bjoern Rabenstein wrote: > > On 24.08.20 09:35, Brian Brazil wrote: > > > On Sun, 23 Aug 2020 at 16:56, Julien Pivotto > > > wrote: > > > > > > On 23 Aug 12:17, Brian Brazil wrote: > > > > On Sun, 23 Aug 2020 at 12:08, Julien Pivotto > > > > > > > wrote: > > > > > > > > > Dear developers, > > > > > > > > > > Prometheus has an experimental api V2 in GRPC, which does not get > > > any > > > > > updates. In addition, it is marked experimental, so we could drop > > > it in > > > > > minor releases. > > > > > > > > > > The code itself is quite small, there is (to my knowledge) no > > > > > documentation and no tests. > > > > > > > > > > The feature surface seems to be really small, with only > > > admin-specific > > > > > API's implemented. > > > > > > > > > > At this point, I'd like to see if we could remove that code, and > > > also > > > > > the cmux library we use, which did not get any updated in 2 > > > years, and > > > > > is incompatible with recent releases of GRPC. > > > > > > > > > > > > The plan of record is to remove it whenever 3.0 is released, > > > however if > > > > it's causing us hassle I see no issue with removing it sooner. > > > > > > 1. CMUX > > > > > > That cmux library is using the helloworld package from grpc which is > > > gone. > > > https://github.com/soheilhy/cmux/blob/ > > > 8a8ea3c53959009183d7914522833c1ed8835020/example_test.go#L32 > > > > > > Additionally, the 2.21 fixes some misconfiguration of the cmux that > > > makes prometheus hang on SIGTERM. I hope that the mitigation we have > > > is > > > complete, but the library lacks features like clean shutdown: > > > https://github.com/soheilhy/cmux/issues/39 > > > > > > At the end the repo seems abandoned since 2 years. > > > > > > 2. GRPC > > > > > > We can not update GRPC, partly because of cmux, partly because we are > > > friendly with downstream, cortex, which depends on etcd, which > > > requires an old version of grpc. > > > > > > This is blocking an update of google.golang.org/api, which is used > > > in our GCE discovery. > > > > > > > > > Those sound like good enough reasons to me. > > > > +1 > > Do we want to add in the 2.21 release notes that we will remove it in > 2.22? Sounds good to me. > > -- > Julien Pivotto > @roidelapluie > > -- > You received this message because you are subscribed to the Google Groups > "Prometheus Developers" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to prometheus-developers+unsubscr...@googlegroups.com. > To view this discussion on the web visit > https://groups.google.com/d/msgid/prometheus-developers/20200827144020.GA601189%40oxygen. > -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/CAM6RFu7R_JWv4b11QnEB1AwhFKFTZgWKBG6N6iHJ6arEMbe4cA%40mail.gmail.com.
Re: [prometheus-developers] Remove /api/v2 of Prometheus
On 27.08.20 16:40, Julien Pivotto wrote: > > Do we want to add in the 2.21 release notes that we will remove it in > 2.22? Good idea. But make clear that this is a weird API that probably next to nobody is actually using. Along the lines: "In the unlikely case that you use the HTTP API v2 (list the endpoint), please note that we will remove this experimental API in the next minor release 2.22." -- Björn Rabenstein [PGP-ID] 0x851C3DA17D748D03 [email] bjo...@rabenste.in -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/20200827144238.GK2356%40jahnn.
Re: [prometheus-developers] Remove /api/v2 of Prometheus
On 27 Aug 16:30, Bjoern Rabenstein wrote: > On 24.08.20 09:35, Brian Brazil wrote: > > On Sun, 23 Aug 2020 at 16:56, Julien Pivotto > > wrote: > > > > On 23 Aug 12:17, Brian Brazil wrote: > > > On Sun, 23 Aug 2020 at 12:08, Julien Pivotto > > > > > wrote: > > > > > > > Dear developers, > > > > > > > > Prometheus has an experimental api V2 in GRPC, which does not get > > any > > > > updates. In addition, it is marked experimental, so we could drop > > it in > > > > minor releases. > > > > > > > > The code itself is quite small, there is (to my knowledge) no > > > > documentation and no tests. > > > > > > > > The feature surface seems to be really small, with only > > admin-specific > > > > API's implemented. > > > > > > > > At this point, I'd like to see if we could remove that code, and > > also > > > > the cmux library we use, which did not get any updated in 2 years, > > and > > > > is incompatible with recent releases of GRPC. > > > > > > > > > The plan of record is to remove it whenever 3.0 is released, however > > if > > > it's causing us hassle I see no issue with removing it sooner. > > > > 1. CMUX > > > > That cmux library is using the helloworld package from grpc which is > > gone. > > https://github.com/soheilhy/cmux/blob/ > > 8a8ea3c53959009183d7914522833c1ed8835020/example_test.go#L32 > > > > Additionally, the 2.21 fixes some misconfiguration of the cmux that > > makes prometheus hang on SIGTERM. I hope that the mitigation we have is > > complete, but the library lacks features like clean shutdown: > > https://github.com/soheilhy/cmux/issues/39 > > > > At the end the repo seems abandoned since 2 years. > > > > 2. GRPC > > > > We can not update GRPC, partly because of cmux, partly because we are > > friendly with downstream, cortex, which depends on etcd, which > > requires an old version of grpc. > > > > This is blocking an update of google.golang.org/api, which is used > > in our GCE discovery. > > > > > > Those sound like good enough reasons to me. > > +1 Do we want to add in the 2.21 release notes that we will remove it in 2.22? -- Julien Pivotto @roidelapluie -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/20200827144020.GA601189%40oxygen.
Re: [prometheus-developers] Remove /api/v2 of Prometheus
On 24.08.20 09:35, Brian Brazil wrote: > On Sun, 23 Aug 2020 at 16:56, Julien Pivotto > wrote: > > On 23 Aug 12:17, Brian Brazil wrote: > > On Sun, 23 Aug 2020 at 12:08, Julien Pivotto > > > wrote: > > > > > Dear developers, > > > > > > Prometheus has an experimental api V2 in GRPC, which does not get any > > > updates. In addition, it is marked experimental, so we could drop it > in > > > minor releases. > > > > > > The code itself is quite small, there is (to my knowledge) no > > > documentation and no tests. > > > > > > The feature surface seems to be really small, with only admin-specific > > > API's implemented. > > > > > > At this point, I'd like to see if we could remove that code, and also > > > the cmux library we use, which did not get any updated in 2 years, and > > > is incompatible with recent releases of GRPC. > > > > > > The plan of record is to remove it whenever 3.0 is released, however if > > it's causing us hassle I see no issue with removing it sooner. > > 1. CMUX > > That cmux library is using the helloworld package from grpc which is > gone. > https://github.com/soheilhy/cmux/blob/ > 8a8ea3c53959009183d7914522833c1ed8835020/example_test.go#L32 > > Additionally, the 2.21 fixes some misconfiguration of the cmux that > makes prometheus hang on SIGTERM. I hope that the mitigation we have is > complete, but the library lacks features like clean shutdown: > https://github.com/soheilhy/cmux/issues/39 > > At the end the repo seems abandoned since 2 years. > > 2. GRPC > > We can not update GRPC, partly because of cmux, partly because we are > friendly with downstream, cortex, which depends on etcd, which > requires an old version of grpc. > > This is blocking an update of google.golang.org/api, which is used > in our GCE discovery. > > > Those sound like good enough reasons to me. +1 -- Björn Rabenstein [PGP-ID] 0x851C3DA17D748D03 [email] bjo...@rabenste.in -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/20200827143000.GH2356%40jahnn.
Re: [prometheus-developers] Remove /api/v2 of Prometheus
On Sun, 23 Aug 2020 at 16:56, Julien Pivotto wrote: > On 23 Aug 12:17, Brian Brazil wrote: > > On Sun, 23 Aug 2020 at 12:08, Julien Pivotto > > > wrote: > > > > > Dear developers, > > > > > > Prometheus has an experimental api V2 in GRPC, which does not get any > > > updates. In addition, it is marked experimental, so we could drop it in > > > minor releases. > > > > > > The code itself is quite small, there is (to my knowledge) no > > > documentation and no tests. > > > > > > The feature surface seems to be really small, with only admin-specific > > > API's implemented. > > > > > > At this point, I'd like to see if we could remove that code, and also > > > the cmux library we use, which did not get any updated in 2 years, and > > > is incompatible with recent releases of GRPC. > > > > > > The plan of record is to remove it whenever 3.0 is released, however if > > it's causing us hassle I see no issue with removing it sooner. > > 1. CMUX > > That cmux library is using the helloworld package from grpc which is > gone. > > https://github.com/soheilhy/cmux/blob/8a8ea3c53959009183d7914522833c1ed8835020/example_test.go#L32 > > Additionally, the 2.21 fixes some misconfiguration of the cmux that > makes prometheus hang on SIGTERM. I hope that the mitigation we have is > complete, but the library lacks features like clean shutdown: > https://github.com/soheilhy/cmux/issues/39 > > At the end the repo seems abandoned since 2 years. > > 2. GRPC > > We can not update GRPC, partly because of cmux, partly because we are > friendly with downstream, cortex, which depends on etcd, which > requires an old version of grpc. > > This is blocking an update of google.golang.org/api, which is used > in our GCE discovery. > Those sound like good enough reasons to me. -- Brian Brazil www.robustperception.io -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/CAHJKeLoUDDCM8eQCez206ccrWj%3Dpb%3DQE-ibR0NLFyEgADbRMFw%40mail.gmail.com.
Re: [prometheus-developers] Remove /api/v2 of Prometheus
On 23 Aug 12:17, Brian Brazil wrote: > On Sun, 23 Aug 2020 at 12:08, Julien Pivotto > wrote: > > > Dear developers, > > > > Prometheus has an experimental api V2 in GRPC, which does not get any > > updates. In addition, it is marked experimental, so we could drop it in > > minor releases. > > > > The code itself is quite small, there is (to my knowledge) no > > documentation and no tests. > > > > The feature surface seems to be really small, with only admin-specific > > API's implemented. > > > > At this point, I'd like to see if we could remove that code, and also > > the cmux library we use, which did not get any updated in 2 years, and > > is incompatible with recent releases of GRPC. > > > The plan of record is to remove it whenever 3.0 is released, however if > it's causing us hassle I see no issue with removing it sooner. 1. CMUX That cmux library is using the helloworld package from grpc which is gone. https://github.com/soheilhy/cmux/blob/8a8ea3c53959009183d7914522833c1ed8835020/example_test.go#L32 Additionally, the 2.21 fixes some misconfiguration of the cmux that makes prometheus hang on SIGTERM. I hope that the mitigation we have is complete, but the library lacks features like clean shutdown: https://github.com/soheilhy/cmux/issues/39 At the end the repo seems abandoned since 2 years. 2. GRPC We can not update GRPC, partly because of cmux, partly because we are friendly with downstream, cortex, which depends on etcd, which requires an old version of grpc. This is blocking an update of google.golang.org/api, which is used in our GCE discovery. -- Julien Pivotto @roidelapluie -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/20200823155604.GA235533%40oxygen.
Re: [prometheus-developers] Remove /api/v2 of Prometheus
On Sun, 23 Aug 2020 at 12:08, Julien Pivotto wrote: > Dear developers, > > Prometheus has an experimental api V2 in GRPC, which does not get any > updates. In addition, it is marked experimental, so we could drop it in > minor releases. > > The code itself is quite small, there is (to my knowledge) no > documentation and no tests. > > The feature surface seems to be really small, with only admin-specific > API's implemented. > > At this point, I'd like to see if we could remove that code, and also > the cmux library we use, which did not get any updated in 2 years, and > is incompatible with recent releases of GRPC. The plan of record is to remove it whenever 3.0 is released, however if it's causing us hassle I see no issue with removing it sooner. -- Brian Brazil www.robustperception.io -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/CAHJKeLrSRcYxqNghwVqnkg9Py4eOqvZ_XPfcbZ-6pxLUAJUurg%40mail.gmail.com.
[prometheus-developers] Remove /api/v2 of Prometheus
Dear developers, Prometheus has an experimental api V2 in GRPC, which does not get any updates. In addition, it is marked experimental, so we could drop it in minor releases. The code itself is quite small, there is (to my knowledge) no documentation and no tests. The feature surface seems to be really small, with only admin-specific API's implemented. At this point, I'd like to see if we could remove that code, and also the cmux library we use, which did not get any updated in 2 years, and is incompatible with recent releases of GRPC. -- Julien Pivotto @roidelapluie -- You received this message because you are subscribed to the Google Groups "Prometheus Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to prometheus-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/prometheus-developers/20200823110803.GA118132%40oxygen.
