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.

Sign up for GitHub

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

Jump to bottom

Metadata: Updating metadata server version from 0.14.0 to v0.21.0 #2901

Merged
merged 1 commit into from
Jan 27, 2020
Merged

Metadata: Updating metadata server version from 0.14.0 to v0.21.0 #2901

merged 1 commit into from
Jan 27, 2020

Conversation

dushyanthsc
Copy link
Contributor

@dushyanthsc dushyanthsc commented Jan 23, 2020 •
edited by IronPan
Loading

Change to bump the metadata version to get the latest updates in MLMD.

This change is Reviewable

@dushyanthsc
Copy link
Contributor Author

/assign @neuromage @Ark-kun @jiaxiao Zheng

@k8s-ci-robot
Copy link
Contributor

@dushyanthsc: GitHub didn't allow me to assign the following users: Jiaxiao, Zheng.

Note that only kubeflow members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

/assign @neuromage @Ark-kun @jiaxiao Zheng

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.

@dushyanthsc
Copy link
Contributor Author

/assign @numerology

@Ark-kun
Copy link
Contributor

Ark-kun commented Jan 23, 2020

/retest

@rmgogogo
Copy link
Contributor

/lgtm
per email, as long as it can work well with our preloaded TFX sample

@neuromage
Copy link
Contributor

/lgtm
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: neuromage

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

The pull request process is described 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

@dushyanthsc
Copy link
Contributor Author

/retest

@dushyanthsc
Copy link
Contributor Author

/retest kubeflow-pipeline-sample-test

1 similar comment
@dushyanthsc
Copy link
Contributor Author

/retest kubeflow-pipeline-sample-test

@dushyanthsc dushyanthsc force-pushed the dushyanthsc/metadata-0-21 branch from 89fe7fc to 1c6e477 Compare January 24, 2020 17:45
@k8s-ci-robot k8s-ci-robot removed the lgtm label Jan 24, 2020
@numerology
Copy link

/lgtm

@dushyanthsc
Copy link
Contributor Author

/retest kubeflow-pipeline-e2e-test

@dushyanthsc
Copy link
Contributor Author

/retest Travis CI - Pull Request

@dushyanthsc
Copy link
Contributor Author

/retest kubeflow-pipeline-e2e-test

1 similar comment
@dushyanthsc
Copy link
Contributor Author

/retest kubeflow-pipeline-e2e-test

@numerology
Copy link

/retest

1 similar comment
@numerology
Copy link

/retest

@numerology
Copy link

/retest

1 similar comment
@numerology
Copy link

/retest

@numerology
Copy link

Interesting... Looks like there's some GKE issues

@dushyanthsc
Copy link
Contributor Author

/retest kubeflow-pipeline-e2e-test

@Ark-kun
Copy link
Contributor

Ark-kun commented Jan 24, 2020

++ gcloud beta container clusters create e2e-7fa66a8-18452 --num-nodes=2 --machine-type=n1-standard-8 --enable-autoscaling --max-nodes=8 --min-nodes=2 --identity-namespace=ml-pipeline-test.svc.id.goog --cluster-version=1.14.8-gke.17

ERROR: (gcloud.beta.container.clusters.create) ResponseError: code=400, message=Master version "1.14.8-gke.17" is unsupported.

@dushyanthsc dushyanthsc force-pushed the dushyanthsc/metadata-0-21 branch from 1c6e477 to 9a68c47 Compare January 24, 2020 21:21
@k8s-ci-robot k8s-ci-robot removed the lgtm label Jan 24, 2020
@numerology
Copy link

/lgtm
from my side.

@dushyanthsc dushyanthsc force-pushed the dushyanthsc/metadata-0-21 branch from 9a68c47 to 7a25327 Compare January 24, 2020 22:00
@k8s-ci-robot k8s-ci-robot removed the lgtm label Jan 24, 2020
@dushyanthsc
Metadata: Updating metadata server version from 0.14.0 to v0.21.0
dedb2dc 
Change to bump the metadata version to get the latest updates in MLMD.
@dushyanthsc dushyanthsc force-pushed the dushyanthsc/metadata-0-21 branch from 7a25327 to dedb2dc Compare January 27, 2020 17:04
@numerology
Copy link

/lgtm

@k8s-ci-robot k8s-ci-robot merged commit 469e719 into kubeflow:master Jan 27, 2020
Jeffwan pushed a commit to Jeffwan/pipelines that referenced this pull request Dec 9, 2020
@dushyanthsc @Jeffwan
Metadata: Updating metadata server version from 0.14.0 to v0.21.0 (ku…
f110090 
…beflow#2901)

Change to bump the metadata version to get the latest updates in MLMD.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@numerology numerology numerology left review comments

@Ark-kun Ark-kun Awaiting requested review from Ark-kun

@IronPan IronPan Awaiting requested review from IronPan

Assignees

@neuromage neuromage

@Ark-kun Ark-kun

@rmgogogo rmgogogo

@numerology numerology

Labels
approved lgtm size/S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@dushyanthsc @k8s-ci-robot @Ark-kun @rmgogogo @neuromage @numerology