Skip to content
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

Katib updates #11

Merged
merged 6 commits into from
Sep 26, 2024
Merged

Katib updates #11

merged 6 commits into from
Sep 26, 2024

Conversation

MaxKavun
Copy link
Collaborator

@MaxKavun MaxKavun commented Sep 24, 2024

Which issue is resolved by this Pull Request:
Resolves #

Description of your changes:
user roles update and values fixes

Checklist:

  • Unit tests pass:
    Make sure you have installed kustomize == 5.2.1+
    1. make generate-changed-only
    2. make test

@MaxKavun MaxKavun marked this pull request as ready for review September 24, 2024 12:29
@MaxKavun MaxKavun requested a review from kromanow94 September 24, 2024 12:29
@kromanow94
Copy link
Owner

@MaxKavun can you investigate this error? I got it using the quickstart.helm.local.sh script:

Error: 1 error occurred:
        * ClusterRole in version "v1" cannot be handled as a ClusterRole: json: cannot unmarshal string into Go struct field LabelSelector.aggregationRule.clusterRoleSelectors.matchLabels of type map[string]string

@MaxKavun
Copy link
Collaborator Author

@kromanow94 that was tested and fixed

@kromanow94 kromanow94 merged commit 03a8567 into kromanow94:katib Sep 26, 2024
1 check passed
MaxKavun added a commit that referenced this pull request Sep 27, 2024
* Katib manifests

* updated katib cert/issuer templating

* updated katib webhook templating

* Update Chart.yaml

* sync work

* add katib example experiment

* Katib updates (#11)

* Update cluster roles

* Fix a few indentations

* fix yaml complaints

* Add condition to execute script only on primary node

* Update template name and fix the bug

* Remove check on script since it's only for replication architecture; fix secret refences

* Fix lint

* Add maintainers

* Rename file to appropriate format

---------

Co-authored-by: Corey Ricketts <cricketts@ncl.com>
Co-authored-by: Corey Ricketts <495237+doncorsean@users.noreply.github.com>
Co-authored-by: Max <maxkavun@outlook.com>
Co-authored-by: Maksim Kavun <maksim.kavun@contractors.roche.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants