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

[DO NOT MERGE] This PR is to let us test the feature-serverside-apply branch #62554

Open
wants to merge 171 commits into
base: master
from

Conversation

@lavalamp
Copy link
Member

lavalamp commented Apr 13, 2018

DO NOT MERGE
DO NOT MERGE
DO NOT MERGE

We can write /test commands below to verify the feature branch continues to work. Leave this PR open indefinitely.

@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Apr 13, 2018

@lavalamp: Adding do-not-merge/release-note-label-needed because the release note process has not been followed.

One of the following labels is required "release-note", "release-note-action-required", or "release-note-none".
Please see: https://git.k8s.io/community/contributors/devel/pull-requests.md#write-release-notes-if-needed.

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.

@apelisse

This comment has been minimized.

Copy link
Member

apelisse commented Apr 13, 2018

Each rebase like this should show exactly how we've diverged from master. I think it's useful. Thanks!

@BenTheElder

This comment has been minimized.

Copy link
Member

BenTheElder commented Apr 13, 2018

/test all

@BenTheElder

This comment has been minimized.

Copy link
Member

BenTheElder commented Apr 13, 2018

hmm yeah, so I mentioned this offline to @lavalamp but:
GitHub APIs have us reporting results to commits rather than PRs, so the best place to see test results from this at the moment is https://k8s-gubernator.appspot.com/pr/62554

apelisse and others added some commits Apr 9, 2018

Add "LastApplied" field in ObjectMeta
This new field will keep track of users/controllers/workflow-id intent
when they apply something. This is part of the server-side apply.
Merge pull request #62498 from apelisse/feature-get-apply
FEATURE-BRANCH: Serverside-Apply: Add new MetaObject field

lavalamp and others added some commits Apr 17, 2018

Merge pull request #62755 from kubernetes/master
[feature branch] sync w/ master
Merge pull request #62809 from kubernetes/master
[feature branch] sync w/ master
Merge pull request #62814 from lavalamp/apply-1
Automatic merge from submit-queue.

add changelog for the feature branch

**What this PR does / why we need it**:

Implement suggestion from sync meeting from a while ago.

**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*:
Fixes #

**Special notes for your reviewer**:

**Release note**:

```release-note
NONE
```
@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Apr 24, 2018

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


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. I understand the commands that are listed here.

apelisse and others added some commits Dec 19, 2018

Move apply test to test/integration/apiserver/apply
Move them out from the apiserver test into a specific directory so that
you can just run these if needed (without having to select using go
-test.run).
Merge pull request #72660 from jennybuckley/sync-with-master-6
[feature branch] Sync with master

kwiesmueller and others added some commits Jan 4, 2019

revert error toleration in FieldManager
fix return in create.go

revert and remove toleration + flag

update godeps
Merge pull request #72281 from apelisse/add-and-move-apply-tests
[FEATURE-SERVERSIDE-APPLY] Add and move apply tests
Merge pull request #72755 from jennybuckley/scheme-models-take-2
[feature branch] [WIP] Support cross group conversion in apply
Merge pull request #72576 from kwiesmueller/feature-serverside-apply
remove field manager error toleration
Merge pull request #72792 from jennybuckley/noop-conversion
[feature branch] skip conversion to same version
@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Jan 11, 2019

@lavalamp: PR needs rebase.

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.

jennybuckley and others added some commits Jan 14, 2019

Merge pull request #72899 from jennybuckley/update-smd
[feature branch] Update structured-merge-diff version
@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Jan 15, 2019

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lavalamp

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

apelisse and others added some commits Jan 15, 2019

Merge pull request #72945 from apelisse/add-kevin-reviewer
Add kwiesmueller, jennybuckley, apelisse as reviewiers for apiserver
@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Jan 16, 2019

@lavalamp: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
pull-kubernetes-bazel-build b926b82 link /test pull-kubernetes-bazel-build
pull-kubernetes-bazel-test b926b82 link /test pull-kubernetes-bazel-test
pull-kubernetes-godeps b926b82 link /test pull-kubernetes-godeps
pull-kubernetes-integration b926b82 link /test pull-kubernetes-integration
pull-kubernetes-e2e-kops-aws b926b82 link /test pull-kubernetes-e2e-kops-aws
pull-kubernetes-e2e-gce-device-plugin-gpu b926b82 link /test pull-kubernetes-e2e-gce-device-plugin-gpu
pull-kubernetes-node-e2e b926b82 link /test pull-kubernetes-node-e2e
pull-kubernetes-verify b926b82 link /test pull-kubernetes-verify
pull-kubernetes-kubemark-e2e-gce-big b926b82 link /test pull-kubernetes-kubemark-e2e-gce-big
pull-kubernetes-typecheck b926b82 link /test pull-kubernetes-typecheck
pull-kubernetes-e2e-gce b926b82 link /test pull-kubernetes-e2e-gce
pull-kubernetes-e2e-gce-100-performance b926b82 link /test pull-kubernetes-e2e-gce-100-performance

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment