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

add installation guidelines to issue template #2846

Conversation

hansinikarunarathne
Copy link
Member

Pull Request Template for Kubeflow manifests Issues

✏️ A brief description of the changes

I installation guidelines to issue template

✅ Contributor checklist


You can join the CNCF Slack and access our meetings at the Kubeflow Community website. Our channel on the CNCF Slack is here #kubeflow-platform.

Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign kimwnasptd for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow bot added size/L and removed size/XS labels Aug 17, 2024
.github/ISSUE_TEMPLATE/issue-report.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/issue-report.yml Outdated Show resolved Hide resolved
options:
- master
- 1.9
- 1.8
Copy link
Member

@juliusvonkohout juliusvonkohout Aug 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drop 1.8, we do not support it anymore.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no not fixed, you removed 1.9 instead of 1.8

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, what do I need to drop 1.8 or 1.9? In the above you have mentioned dropping 1.9

Copy link
Member

@juliusvonkohout juliusvonkohout Aug 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right, now it says correctly drop 1.8

Copy link
Member

@juliusvonkohout juliusvonkohout Aug 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So please drop 1.8 and keep 1.9

.github/ISSUE_TEMPLATE/issue-report.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/issue-report.yml Outdated Show resolved Hide resolved
1. Try this ...
2. Then do this ...
validations:
- label: Did you follow the installation guide https://github.com/kubeflow/manifests?tab=readme-ov-file ?
Copy link
Member

@juliusvonkohout juliusvonkohout Aug 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

- label: Did you follow the [Kubeflow installation guide](https://github.com/kubeflow/manifests/blob/master/README.md) ?

- type: markdown
attributes:
value: |
Follow the [Kubeflow Guidelines](https://github.com/kubeflow/manifests/blob/master/README.md) for installation.
Copy link
Member

@juliusvonkohout juliusvonkohout Aug 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Follow the [Kubeflow installation guideline](https://github.com/kubeflow/manifests/blob/master/README.md).

@google-oss-prow google-oss-prow bot added size/XL and removed size/L labels Aug 26, 2024
hansinikarunarathne and others added 10 commits August 26, 2024 18:46
Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
* Added pipelines to PSS test and fixed typo

Signed-off-by: biswajit-9776 <[email protected]>

* Added dynamic namespaces to PSS test

Signed-off-by: biswajit-9776 <[email protected]>

---------

Signed-off-by: biswajit-9776 <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
* Training operator  CICD improvements (kubeflow#2779)

* Add the networkpolicies

Signed-off-by: juliusvonkohout <[email protected]>

* rework the training operator tests

Signed-off-by: juliusvonkohout <[email protected]>

* fix the comments

Signed-off-by: juliusvonkohout <[email protected]>

* fix filename

Signed-off-by: juliusvonkohout <[email protected]>

* try to fix the permissions

Signed-off-by: juliusvonkohout <[email protected]>

* try to fix the permissions

Signed-off-by: juliusvonkohout <[email protected]>

* change to the user namespace

Signed-off-by: juliusvonkohout <[email protected]>

* update the image to rc.1

Signed-off-by: juliusvonkohout <[email protected]>

* fixes

Signed-off-by: juliusvonkohout <[email protected]>

* fixes

Signed-off-by: juliusvonkohout <[email protected]>

* fixes

Signed-off-by: juliusvonkohout <[email protected]>

* fixes

Signed-off-by: juliusvonkohout <[email protected]>

* fixes

Signed-off-by: juliusvonkohout <[email protected]>

---------

Signed-off-by: juliusvonkohout <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* create seperate file for install_KinD_create_KinD_cluster_install_kustomize.sh to avoid code repitition

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* make sh file executable

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* remove reduntant codes

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* combine kind, kuztomize and kind-cluster create sh files into one file

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* fix issues with combined file

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* fix configurations issues

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* fix linting issues in workflow files

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* fix trailing issues in workflow files

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* fix trailing issues in workflow file bentoml

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* made tests/gh-actions/install_KinD_create_KinD_cluster_install_kustomize.sh executable and removed chmod command from the workflow files

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* made modifications in workflow files

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* deleted redundant files from tests folder

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* Add kustomize installation file

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* made install_kustomize.sh file executable

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* made a fix

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* fix issue with linting of YAML files (kubeflow#2825)

* fix issue with linting of YAML files

Signed-off-by: hansinikarunarathne <[email protected]>

* delete a file to check the functionality

Signed-off-by: hansinikarunarathne <[email protected]>

* Add the after checking the functionality

Signed-off-by: hansinikarunarathne <[email protected]>

* Add folders to yaml linting

Signed-off-by: hansinikarunarathne <[email protected]>

* check linting issues

Signed-off-by: hansinikarunarathne <[email protected]>

* check linting issues

Signed-off-by: hansinikarunarathne <[email protected]>

* check linting changes

Signed-off-by: hansinikarunarathne <[email protected]>

* check linting functionality by chnaging different files

Signed-off-by: hansinikarunarathne <[email protected]>

* made some fixes in linting YAML file

Signed-off-by: hansinikarunarathne <[email protected]>

* made some fixes in linting YAML file

Signed-off-by: hansinikarunarathne <[email protected]>

* made some fixes in linting YAML file and check the functionality

Signed-off-by: hansinikarunarathne <[email protected]>

* made some fixes in linting YAML file and check the functionality

Signed-off-by: hansinikarunarathne <[email protected]>

* Add github, hack and tests folders also for linting

Signed-off-by: hansinikarunarathne <[email protected]>

* Check functionality by deleting sh from hack and yaml file common

Signed-off-by: hansinikarunarathne <[email protected]>

* Added deleted files

Signed-off-by: hansinikarunarathne <[email protected]>

---------

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* Expose Ray and Seldon to example kustomization.yaml file (kubeflow#2834)

* Expose Ray and Seldon to example kustomization.yaml file

Signed-off-by: hansinikarunarathne <[email protected]>

* Add internal documentations for ray and seldon

Signed-off-by: hansinikarunarathne <[email protected]>

---------

Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* Increase the time out of notebook and katib test

Signed-off-by: hansinikarunarathne <[email protected]>

* increase the timout time

Signed-off-by: hansinikarunarathne <[email protected]>

* increase timeout of the create pipeline Run from Kubeflow Notebook to 2400s

Signed-off-by: hansinikarunarathne <[email protected]>

* Add code to calculate time taken to pull a docker image

Signed-off-by: hansinikarunarathne <[email protected]>

* fix syntax error in yaml file

Signed-off-by: hansinikarunarathne <[email protected]>

* increase pipeline time

Signed-off-by: hansinikarunarathne <[email protected]>

* remove one step in action

Signed-off-by: hansinikarunarathne <[email protected]>

* Added tests to tests/gh-actions to enable baseline and restricted PSS (kubeflow#2819)

* Patched PSS labels to multi_tenancy

Signed-off-by: biswajit-9776 <[email protected]>

* Added script in gh-actions to patch PSS/static/baseline/pacthes

Signed-off-by: biswajit-9776 <[email protected]>

* Added PSS scripts for both baseline and restricted labels of static namespaces and renamed directories

Signed-off-by: biswajit-9776 <[email protected]>

* Added tests to enable PSS in gh-actions

Signed-off-by: biswajit-9776 <[email protected]>

* Added workflow test for PSS labels

Signed-off-by: biswajit-9776 <[email protected]>

* Fixed indentation

Signed-off-by: biswajit-9776 <[email protected]>

---------

Signed-off-by: biswajit-9776 <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>

* add kind cluster step into pss test yaml file

Signed-off-by: hansinikarunarathne <[email protected]>

* change time pot time

Signed-off-by: hansinikarunarathne <[email protected]>

---------

Signed-off-by: juliusvonkohout <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: biswajit-9776 <[email protected]>
Co-authored-by: Julius von Kohout <[email protected]>
Co-authored-by: biswajit-9776 <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
* renable more parts of the dex test

Signed-off-by: juliusvonkohout <[email protected]>

* fix dex-login-test (kubeflow#2830)

* fix dex-login-test:
- set executable permission for python script
- set shebang line in python script

Signed-off-by: Tom Zaspel <[email protected]>

* Add all required componentes which have to be installed to test dex login

Signed-off-by: Tom Zaspel <[email protected]>

* Add kustomizeation file to deploy all required stuff to test dex login

Signed-off-by: Tom Zaspel <[email protected]>

* Add statuscode response in dex login

Signed-off-by: Tom Zaspel <[email protected]>

---------

Signed-off-by: Tom Zaspel <[email protected]>
Signed-off-by: Tom Zaspel <[email protected]>

---------

Signed-off-by: juliusvonkohout <[email protected]>
Signed-off-by: Tom Zaspel <[email protected]>
Signed-off-by: Tom Zaspel <[email protected]>
Co-authored-by: Tom Zaspel <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
* Update kubeflow/kubeflow manifests from v1.9.0

Signed-off-by: juliusvonkohout <[email protected]>

* update synchronization script

Signed-off-by: juliusvonkohout <[email protected]>

---------

Signed-off-by: juliusvonkohout <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
Signed-off-by: hansinikarunarathne <[email protected]>
@hansinikarunarathne hansinikarunarathne force-pushed the feature-add-installation-guidelines-to-README-file branch from c73cfdd to 0ee68ec Compare August 26, 2024 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants