Update kubeflow/spark-operator manifests to 2.2.0#3146
Conversation
Signed-off-by: juliusvonkohout <45896133+juliusvonkohout@users.noreply.github.com>
Signed-off-by: Julius von Kohout <45896133+juliusvonkohout@users.noreply.github.com>
|
@akagami-harsh do you mind checking, whether the seccomprofile in the securitycontext is now upstream ? |
| | Kubeflow Pipelines | apps/pipeline/upstream | [2.5.0](https://github.com/kubeflow/pipelines/tree/2.5.0/manifests/kustomize) | | ||
| | Kubeflow Model Registry | apps/model-registry/upstream | [v0.2.18](https://github.com/kubeflow/model-registry/tree/v0.2.18/manifests/kustomize) | | ||
| | Spark Operator | apps/spark/spark-operator | [2.1.1](https://github.com/kubeflow/spark-operator/tree/v2.1.1) | | ||
| | Spark Operator | apps/spark/spark-operator | 2.2.0) | |
There was a problem hiding this comment.
@kunal-511 something went wrong with the synchronization script.
There was a problem hiding this comment.
the issue is that sed command in the script is only looking for the version number pattern and doesn't account for the markdown link format.
There was a problem hiding this comment.
need to update it in script to account for the link in md format
Yes they are upstream : https://github.com/kubeflow/spark-operator/blame/08dfbc5fc958b91d3be4d79a0848b44b72f7d98b/charts/spark-operator-chart/values.yaml#L184C2-L184C2 |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: juliusvonkohout The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Julius von Kohout <45896133+juliusvonkohout@users.noreply.github.com>
|
/lgtm |
4 participants
Lets test whether the security context patches are still needed.