search

scalyr / scalyr-tool

Command-line tool for accessing Scalyr services
Apache License 2.0
56 stars 37 forks source link

Bring timeseries-query up-to-date to not require a timeseries id #14

Closed jjhart closed 7 years ago

jjhart commented 7 years ago

@steve-scalyr @code-sauce @czerwingithub Hi all, it's my annual "do some python for fun & profit" moment.

We've been telling our customers recently to start using the "new" form of the timeseries query, where they don't have to first call create-timeseries (and persist the returned id) but instead they can just invoke with the same arguments as numeric-query, and we'll appropriately create the right timeseries for them on the backend.

I thought that scalyr-tool supported this already, so I started by rewriting the README file. Turns out I was wrong, so I had to update the tool also.

Note: doing this in a separate repo just to keep the main (public) repo clean.

code-sauce commented 7 years ago

I lack the overall context, but the overall Python changes look good to me 👍

czerwingithub commented 7 years ago

Yup, looked good to me as well, fwiw

On Wed, Sep 6, 2017 at 5:24 PM, John Hart notifications@github.com wrote:

Merged #14 https://github.com/scalyr/scalyr-tool/pull/14.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/scalyr/scalyr-tool/pull/14#event-1237879821, or mute the thread https://github.com/notifications/unsubscribe-auth/ACcri824GJd16KwVfUgBla9NtGrdf7peks5sfze8gaJpZM4PO_FA .