Mason Malone

Results 66 comments of Mason Malone

Test failures are unrelated and should go away when https://github.com/argoproj/argo-workflows/pull/13660 is merged

Thanks for the report! Here's a slightly simpler workflow to reproduce this discrepancy that doesn't use a `WorkflowTemplate`: ```yaml apiVersion: v1 kind: ConfigMap metadata: name: config-properties labels: workflows.argoproj.io/configmap-type: Parameter data:...

You're probably already aware, but one feature of logrus we need is spying on the logging statements in unit tests using https://pkg.go.dev/github.com/sirupsen/logrus/hooks/test. Example: https://github.com/argoproj/argo-workflows/blob/2fd54884844bb76d760466027afa023c5bfd6b64/util/cmd/cmd_test.go#L145-L147 To replicate that, I think the...

@nicdard, @sarabala1979 , and I met on May 13th to discuss this, and I agreed to update the PR with the result of that converstation. Sorry for the delay! My...

@Joibel Sometimes, but I switched from Homebrew to Nix, and I don't run `make devcontainer-up` on that machine. I suspect almost all the other `TOOL_*` commands are going to have...

FYI: For Argo Workfows, this was fixed in https://github.com/argoproj/argo-workflows/pull/14105. I'm not sure if it's worth documenting this in the README.md, so feel free to close this.

@tico24 You're right. I mentioned in the PR description, but I think the ideal would be to include the `ValidatingAdmissionPolicy` in the release manifests so people can quickly enable this...

I'm looking into this, and I'm having trouble with the "Only show this input box if SSO RBAC Namespace delegation is enabled" requirement. Unless I'm missing something, the UI currently...