Skip to content

fix: remove duplicated %} in kustomization patch

Illia Shestakov requested to merge ishestakov/tutor-contrib-grove:kustomization-patch-fix into main

Description

Fix a typo in kustomization patch. This typo in causing an error when running ./grove deploy <name>. The error is :

=====================================

        Starting the platform

=====================================

kubectl get namespaces example

NAME      STATUS   AGE

example   Active   18h

Namespace already exists: skipping creation.

kubectl apply --kustomize /workspace/instances/example/env --wait --selector app.kubernetes.io/component=volume

error: error converting YAML to JSON: yaml: line 89: could not find expected directive name

Error: Command failed with status 1: kubectl apply --kustomize /workspace/instances/example/env --wait --selector app.kubernetes.io/component=volume

Command failed with status 1: /workspace/instances/example/.venv/bin/tutor k8s launch --non-interactive
Aborted!

Supporting information

Link to other information about the change, such as GitLab issues, GitHub issues, forum discussions. Be sure to check they are publicly readable, or if not, repeat the information here.

Testing instructions

Steps to test the changes:

  1. TBD

Dependencies

List the dependencies required for this change, if any. Do not forget to link grove-template merge request here if that's affected by this change.

Screenshots

If applicable, add screenshots to help explain your feature.

Checklist

If any of the items below is not applicable, do not remove them, but put a check in it.

  • Git commit history is clean
  • Git commits are following conventional-commit practices
  • Unit tests are added/updated
  • Documentation is added/updated

Additional context

Add any other context about the merge request here.

Merge request reports