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
fix(server): Allow field selection for workflow-event endpoint (fixes #3163) #3165
Conversation
…abel in the new labels column
… to take up less real estate.
…ait from labels column
…mponent to keep track of show/hide state. Reformat column widths for readability and change format of labels
…ces duration, and conditions
go.mod
Outdated
@@ -29,7 +29,7 @@ require ( | |||
github.com/go-openapi/swag v0.19.8 // indirect | |||
github.com/go-sql-driver/mysql v1.4.1 | |||
github.com/gogo/protobuf v1.3.1 | |||
github.com/golang/protobuf v1.4.0 | |||
github.com/golang/protobuf v1.3.5 |
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.
I think the changes to go.mod
and go.sum
were unintentional. Could you revert them back by doing git checkout master go.mod go.sum
?
@@ -89,6 +89,7 @@ message WorkflowDeleteResponse { | |||
message WatchWorkflowsRequest { | |||
string namespace = 1; | |||
k8s.io.apimachinery.pkg.apis.meta.v1.ListOptions listOptions = 2; | |||
string fields = 3; |
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.
Where is this field actually used? I don't see it in the Argo Server code
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.
In fact I don't see it used anywhere in List either: https://github.com/rbreeze/argo/blob/162f49559cb4a24eadccb20129b8ce04fb36faca/server/workflow/workflow_server.go#L94
Seems like that was the bug?
Kudos, SonarCloud Quality Gate passed! 0 Bugs |
Checklist:
"fix(controller): Updates such and such. Fixes #1234"
.