-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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-Create-Experiment-button-when-selecting-experiement-#5737 #11060
Add-Create-Experiment-button-when-selecting-experiement-#5737 #11060
Conversation
Signed-off-by: Elay Aharoni (EXT-Nokia) <[email protected]>
Hi @ElayAharoni. Thanks for your PR. I'm waiting for a kubeflow member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/ok-to-test |
Signed-off-by: Elay Aharoni (EXT-Nokia) <[email protected]>
…n-selecting-experiement-#5737' into Add-Create-Experiment-button-when-selecting-experiement-kubeflow#5737 # Conflicts: # frontend/src/pages/NewRunV2.tsx # frontend/src/pages/functional_components/NewExperimentFC.tsx
i fixed the formatting of the code, please if i can have a second review. thanks |
/rerun-all |
…n-selecting-experiement-#5737' into Add-Create-Experiment-button-when-selecting-experiement-kubeflow#5737
…n-selecting-experiement-#5737' into Add-Create-Experiment-button-when-selecting-experiement-kubeflow#5737
thanks, i had one more lint issue that i missed.. can you please retrigger the tests whenever you can? |
/rerun-all @ElayAharoni you may be authorized to retrigger them. I'm not sure. |
/rerun-all |
hi the pr has passed the tests, if i can have another review please. |
hi is there any progress with the review of the pr? please let me know if there is anything i need to do |
please if someone can review the PR when possible. |
…flow#11231) Signed-off-by: Chen Sun <[email protected]>
…flow#11127) Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.5 to 6.5.7. - [Commits](indutny/elliptic@v6.5.5...v6.5.7) --- updated-dependencies: - dependency-name: elliptic dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…n /frontend/server (kubeflow#11056) Bumps [fast-xml-parser](https://github.com/NaturalIntelligence/fast-xml-parser) to 4.4.1 and updates ancestor dependency [@aws-sdk/credential-providers](https://github.com/aws/aws-sdk-js-v3/tree/HEAD/packages/credential-providers). These dependencies need to be updated together. Updates `fast-xml-parser` from 4.2.5 to 4.4.1 - [Release notes](https://github.com/NaturalIntelligence/fast-xml-parser/releases) - [Changelog](https://github.com/NaturalIntelligence/fast-xml-parser/blob/master/CHANGELOG.md) - [Commits](NaturalIntelligence/fast-xml-parser@v4.2.5...v4.4.1) Updates `@aws-sdk/credential-providers` from 3.359.0 to 3.621.0 - [Release notes](https://github.com/aws/aws-sdk-js-v3/releases) - [Changelog](https://github.com/aws/aws-sdk-js-v3/blob/main/packages/credential-providers/CHANGELOG.md) - [Commits](https://github.com/aws/aws-sdk-js-v3/commits/v3.621.0/packages/credential-providers) --- updated-dependencies: - dependency-name: fast-xml-parser dependency-type: indirect - dependency-name: "@aws-sdk/credential-providers" dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [send](https://github.com/pillarjs/send) to 0.19.0 and updates ancestor dependency [express](https://github.com/expressjs/express). These dependencies need to be updated together. Updates `send` from 0.18.0 to 0.19.0 - [Release notes](https://github.com/pillarjs/send/releases) - [Changelog](https://github.com/pillarjs/send/blob/master/HISTORY.md) - [Commits](pillarjs/send@0.18.0...0.19.0) Updates `express` from 4.20.0 to 4.21.0 - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/4.21.0/History.md) - [Commits](expressjs/express@4.20.0...4.21.0) --- updated-dependencies: - dependency-name: send dependency-type: indirect - dependency-name: express dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This commit adds a github workflow that will build v2 images and push them to GHCR. It uses the GITHUB_TOKEN to authenticate and uses docker provisioned github actions to streamline the flow. The workflow also creates attestations for the packages which can be used to verify provenance and integrity. The workflow can be triggered manually or via another workflow call. The latter is to supplement future automation flows. Signed-off-by: Humair Khan <[email protected]>
…flow#11294) * Use kustomize to patch deployments before deploy Signed-off-by: carter.fendley <[email protected]> * Relocate scripts / manifests to github directory Signed-off-by: carter.fendley <[email protected]> --------- Signed-off-by: carter.fendley <[email protected]>
…11306) Signed-off-by: Helber Belmiro <[email protected]>
) Moving components that as no maintenance activities in the past two years to contrib folder. Signed-off-by: Chen Sun <[email protected]>
) Signed-off-by: VaniHaripriya <[email protected]>
* fix(tests): remove redundant integration test wf Signed-off-by: droctothorpe <[email protected]> * Add paths Signed-off-by: droctothorpe <[email protected]> * Remove integration-test.sh Signed-off-by: droctothorpe <[email protected]> --------- Signed-off-by: droctothorpe <[email protected]>
@ElayAharoni it's been waiting for some time. Can you rebase? |
* adding chore as a new template Signed-off-by: varodrig <[email protected]> * Update .github/ISSUE_TEMPLATE/CHORE.md Co-authored-by: Helber Belmiro <[email protected]> Signed-off-by: Valentina Rodriguez Sosa <[email protected]> * Update .github/ISSUE_TEMPLATE/CHORE.md Co-authored-by: Helber Belmiro <[email protected]> Signed-off-by: Valentina Rodriguez Sosa <[email protected]> * Update .github/ISSUE_TEMPLATE/CHORE.md Co-authored-by: Helber Belmiro <[email protected]> Signed-off-by: Valentina Rodriguez Sosa <[email protected]> * Update .github/ISSUE_TEMPLATE/CHORE.md Co-authored-by: Helber Belmiro <[email protected]> Signed-off-by: Valentina Rodriguez Sosa <[email protected]> * Update .github/ISSUE_TEMPLATE/CHORE.md Co-authored-by: Helber Belmiro <[email protected]> Signed-off-by: Valentina Rodriguez Sosa <[email protected]> * Update .github/ISSUE_TEMPLATE/CHORE.md Co-authored-by: Helber Belmiro <[email protected]> Signed-off-by: Valentina Rodriguez Sosa <[email protected]> --------- Signed-off-by: varodrig <[email protected]> Signed-off-by: Valentina Rodriguez Sosa <[email protected]> Co-authored-by: Helber Belmiro <[email protected]>
/lgtm @ElayAharoni apologies for the wait, can you rebase? once done I can merge this. |
hi @HumairAK sorry for the delay, i did rebase now |
Signed-off-by: Elay Aharoni (EXT-Nokia) <[email protected]>
Signed-off-by: Elay Aharoni (EXT-Nokia) <[email protected]>
…n-selecting-experiement-#5737' into Add-Create-Experiment-button-when-selecting-experiement-kubeflow#5737 # Conflicts: # frontend/src/pages/NewRunV2.tsx
New changes are detected. LGTM label has been removed. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 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 |
…n-selecting-experiement-#5737' into Add-Create-Experiment-button-when-selecting-experiement-kubeflow#5737
…n-selecting-experiement-#5737' into Add-Create-Experiment-button-when-selecting-experiement-kubeflow#5737 # Conflicts: # frontend/src/pages/NewRunV2.tsx
…n-selecting-experiement-#5737' into Add-Create-Experiment-button-when-selecting-experiement-kubeflow#5737
…n-selecting-experiement-#5737' into Add-Create-Experiment-button-when-selecting-experiement-kubeflow#5737
@ElayAharoni 48 commits. It seems like the rebase was not properly done. |
yeah i did something wrong when rebased it, i will drop this pr and created a new one for this issue.. |
/close |
@hbelmiro: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
feat(frontend): add create experiment button when selecting experiment.