Skip to content

Comments

*: Cut v0.16.0-alpha.0#1646

Merged
mxinden merged 1 commit intoprometheus:masterfrom
mxinden:cut-v0.16.0-alpha.0
Nov 30, 2018
Merged

*: Cut v0.16.0-alpha.0#1646
mxinden merged 1 commit intoprometheus:masterfrom
mxinden:cut-v0.16.0-alpha.0

Conversation

@mxinden
Copy link
Member

@mxinden mxinden commented Nov 30, 2018

Given the long list of changes I would like to start with an alpha release first. Hence this replaces #1553.

Signed-off-by: Max Leonard Inden IndenML@gmail.com

@mxinden
Copy link
Member Author

mxinden commented Nov 30, 2018

@stuartnelson3 do you think we should hold here due to #1643? I would be fine with a patch release once we can reproduce the issue.

@mxinden mxinden changed the title *: Cut v0.16.0-beta.0 *: Cut v0.16.0-alpha.0 Nov 30, 2018
@mxinden mxinden force-pushed the cut-v0.16.0-alpha.0 branch from fc82aa8 to 67d9cb9 Compare November 30, 2018 13:09
@mxinden mxinden mentioned this pull request Nov 30, 2018
@mxinden mxinden added this to the 0.16.0 milestone Nov 30, 2018
Copy link
Member

@simonpasquier simonpasquier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple of nits.

CHANGELOG.md Outdated
* [FEATURE] Introduce OpenAPI generated API v2 (#1352)
* [FEATURE] Lookup parts in strings using regexp.MatchString in templates (#1452)
* [FEATURE] Support image/thumb url in attachment in Slack notifier (#1506)
* [FEATURE] Support custom TLS certificates (#1528)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Support custom TLS certificates for the email notifier (#1528)

CHANGELOG.md Outdated
* [ENHANCEMENT] Add name, value and SlackConfirmationField to action in Slack notifier (#1557)
* [ENHANCEMENT] Show more alert information on silence form and silence view pages (#1601)
* [ENHANCEMENT] Add cluster peers DNS refresh job (#1428)
* [ENHANCEMENT] [amtool] Support assuming first label is alertname in alert query with matchers (#1575)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

more a bugfix?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, this was the described behavior in the tool's help docs, I would say this was a bugfix

Signed-off-by: Max Leonard Inden <IndenML@gmail.com>
@mxinden mxinden force-pushed the cut-v0.16.0-alpha.0 branch from 67d9cb9 to e30c73d Compare November 30, 2018 13:20
@mxinden
Copy link
Member Author

mxinden commented Nov 30, 2018

Just a couple of nits.

Thanks @simonpasquier. Fixed.

@stuartnelson3
Copy link
Contributor

Looks good once it's green

Copy link
Member

@simonpasquier simonpasquier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mxinden mxinden merged commit cb1f78c into prometheus:master Nov 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants