-
Notifications
You must be signed in to change notification settings - Fork 88
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
Render tags on invocations and support them in invocation search + invocation trends. #3963
Conversation
proto/invocation.proto
Outdated
@@ -267,6 +267,9 @@ message InvocationQuery { | |||
|
|||
// The pattern for the targets built (exact match). Ex: "//..." | |||
string pattern = 15; | |||
|
|||
// A plaintext tag for the targets built (exact match). Ex: "my-cool-tag" | |||
string tag = 16; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lame question: would it be a lot harder to support repeated tags here? or a comma separated string?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not really, but i'm out tomorrow so you're getting it monday :P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just kidding, love 2 write software, here it is.
ed9e1dd
to
cbc68ac
Compare
Need to add tags to the executions table in clickhouse as well, will do that separately.
Related issues: N/A