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

Manifests release 1.1 #1252

Closed
1 of 2 tasks
krishnadurai opened this issue Jun 15, 2020 · 18 comments
Closed
1 of 2 tasks

Manifests release 1.1 #1252

krishnadurai opened this issue Jun 15, 2020 · 18 comments

Comments

@krishnadurai
Copy link
Contributor

krishnadurai commented Jun 15, 2020

Opening this issue to track releasing manifests as part of 1.1.

/assign @krishnadurai

@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the labels:

Label Probability
kind/feature 0.78
area/kfctl 0.73

Please mark this comment with 👍 or 👎 to give our bot feedback!
Links: app homepage, dashboard and code for this bot.

@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the labels:

Label Probability
area/kfctl 0.73
kind/feature 0.78

Please mark this comment with 👍 or 👎 to give our bot feedback!
Links: app homepage, dashboard and code for this bot.

@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the labels:

Label Probability
area/engprod 0.74

Please mark this comment with 👍 or 👎 to give our bot feedback!
Links: app homepage, dashboard and code for this bot.

@jlewi
Copy link
Contributor

jlewi commented Jun 22, 2020

Looks like the postsubmit tests are failing. Looks like the TFJob operator tests are out of sync.

kubeflow-manifests-postsubmit-unit-d6a25c6-5684-13a5-1216454708.log.txt

@jlewi
Copy link
Contributor

jlewi commented Jun 22, 2020

@krishnadurai Can you fix the tests and then once the tests are fixed cut the branch?

@jlewi
Copy link
Contributor

jlewi commented Jun 22, 2020

@krishnadurai looks like the tests are still failing

===== ACTUAL END ==========================================
  EXPECTED                                                                ACTUAL
  --------                                                                ------
  apiVersion: admissionregistration.k8s.io/v1beta1                        apiVersion: admissionregistration.k8s.io/v1beta1
  kind: MutatingWebhookConfiguration                                      kind: MutatingWebhookConfiguration
  metadata:                                                               metadata:
    annotations:                                                            annotations:
      cert-manager.io/inject-ca-from: kubeflow/admission-webhook-cert         cert-manager.io/inject-ca-from: kubeflow/admission-webhook-cert
    labels:                                                                 labels:
      app: admission-webhook                                                  app: admission-webhook
      app.kubernetes.io/component: poddefaults                                app.kubernetes.io/component: poddefaults
      app.kubernetes.io/name: poddefaults                                     app.kubernetes.io/name: poddefaults
      kustomize.component: admission-webhook                                  kustomize.component: admission-webhook
    name: admission-webhook-mutating-webhook-configuration                  name: admission-webhook-mutating-webhook-configuration
  webhooks:                                                               webhooks:
  - clientConfig:                                                         - clientConfig:
      caBundle: ""                                                            caBundle: ""
      service:                                                                service:
        name: admission-webhook-service                                         name: admission-webhook-service
        namespace: kubeflow                                                     namespace: kubeflow
        path: /apply-poddefault                                                 path: /apply-poddefault
    name: admission-webhook-deployment.kubeflow.org                         name: admission-webhook-deployment.kubeflow.org
X    rules:                                                                  namespaceSelector:
X    - apiGroups:                                                              matchLabels:
X      - ""                                                                      app.kubernetes.io/part-of: kubeflow-profile
X      apiVersions:                                                          rules:
X      - v1                                                                  - apiGroups:
X      operations:                                                             - ""
X      - CREATE                                                                apiVersions:
X      resources:                                                              - v1
X      - pods                                                                  operations:
X                                                                              - CREATE
X                                                                              resources:
X                                                                              - pods
   TestKustomize: test_util.go:176: Expected not equal to actual
--- FAIL: TestKustomize (5.45s)

Even after #1297; My conjecture is that when you generated the tests you weren't based off of the latest master and as a result you are missing #1284

jlewi pushed a commit to jlewi/manifests that referenced this issue Jun 22, 2020
k8s-ci-robot pushed a commit that referenced this issue Jun 22, 2020
@jlewi
Copy link
Contributor

jlewi commented Jun 22, 2020

Tests are passing on
d771072

We should cut the 1.1 branch on that commit.

@jlewi
Copy link
Contributor

jlewi commented Jun 23, 2020

Branch 1.1. has been cut.

@jlewi
Copy link
Contributor

jlewi commented Jun 29, 2020

@krishnadurai Is there any more work here? Are we just waiting to tag a particular commit as 1.1.0?

@krishnadurai
Copy link
Contributor Author

@Jeffwan had mentioned that he wants to patch in a few changes. Otherwise we look good to go for 1.1.0.

@jlewi
Copy link
Contributor

jlewi commented Jul 10, 2020

Created a Tag from the latest commit
https://github.com/kubeflow/manifests/releases/tag/v1.1-rc.0

@jlewi
Copy link
Contributor

jlewi commented Jul 14, 2020

@Bobgy
Copy link
Contributor

Bobgy commented Jul 15, 2020

@jlewi all required cherry picks for multi-user pipelines are already in. Just double checking we are not publishing the public release right now? I found some issues in kubeflow/pipelines#3241 (comment), and need some time to investigate.

@jlewi
Copy link
Contributor

jlewi commented Jul 20, 2020

@Bobgy We need to get the fixes for the KptFiles in GoogleCloudPlatform/kubeflow-distribution#89

@jlewi
Copy link
Contributor

jlewi commented Jul 21, 2020

On GCP side we got broken by changes in kpt (GoogleCloudPlatform/kubeflow-distribution#89). This is now fixed on master.
Master is now green except for 2 tests which appear to be failing due to problems in the tests
https://k8s-testgrid.appspot.com/sig-big-data#kubeflow-gcp-blueprints-periodic-master

Plan is to get the fixes for GoogleCloudPlatform/kubeflow-distribution#89 cherry-picked into the 1.1 branch and then cut an RC.

@jlewi
Copy link
Contributor

jlewi commented Jul 21, 2020

@jlewi
Copy link
Contributor

jlewi commented Jul 23, 2020

New RC
https://github.com/kubeflow/manifests/releases/tag/v1.1-rc.2

This one is an actual release (tag) and not a branch. This is what we want

@jlewi
Copy link
Contributor

jlewi commented Jul 26, 2020

v1.1.0 has been finalized.
https://github.com/kubeflow/manifests/releases/tag/v1.1.0

Note I had to create a tag and not a release because subsequent changes had been cherry-picked onto the branch and GitHub only allows creating a release from the of a branch.

@jlewi jlewi closed this as completed Jul 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants