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

rebase 1.10.0 #56

Merged

Conversation

@liggitt liggitt mentioned this pull request Mar 31, 2018
@liggitt liggitt force-pushed the rebase-1.10.0 branch 4 times, most recently from 3b3e8bc to 9070269 Compare April 1, 2018 02:12
@liggitt liggitt mentioned this pull request Apr 3, 2018
51 tasks
@liggitt liggitt changed the title WIP - rebase 1.10.0 rebase 1.10.0 Apr 5, 2018
@soltysh
Copy link

soltysh commented Apr 6, 2018

There's this error when running tests:

--- FAIL: TestDeleteEdges_locked (0.00s)
    --- FAIL: TestDeleteEdges_locked/edges_are_deleted,_non-orphans_and_destination_orphans_are_preserved (0.00s)
	Error Trace:	graph_test.go:152
    	Error:      	Not equal: 
    	            	expected: []graph.Node{(*node.namedVertex)(0xc420370210), (*node.namedVertex)(0xc4203701e0), (*node.namedVertex)(0xc4203702a0)}
    	            	actual: []graph.Node{(*node.namedVertex)(0xc420370030), (*node.namedVertex)(0xc420353f50), (*node.namedVertex)(0xc420353f80)}
    	            	
    	            	Diff:
    	            	--- Expected
    	            	+++ Actual
    	            	@@ -1,5 +1,5 @@
    	            	 ([]graph.Node) (len=3) {
    	            	- (*node.namedVertex)(node:node2),
    	            	+ (*node.namedVertex)(configmap:namespace1/configmap1),
    	            	  (*node.namedVertex)(node:node1),
    	            	- (*node.namedVertex)(configmap:namespace1/configmap1)
    	            	+ (*node.namedVertex)(node:node2)
    	            	 }
FAIL
FAIL	k8s.io/kubernetes/plugin/pkg/auth/authorizer/node	0.064s

@liggitt
Copy link
Author

liggitt commented Apr 6, 2018

@soltysh not sure what branch you're looking at, but that's only on master (post-1.10), not in this branch...

@soltysh
Copy link

soltysh commented Apr 6, 2018

@liggitt I'll double check once again when I'm back later today. I swear I run it on the right branch 😉

@soltysh
Copy link

soltysh commented Apr 6, 2018

Yeah, I must have been building something completely different, but what worries me is that the current state is so tightly coupled with origin that it's impossible to build it and test it :/

@liggitt
Copy link
Author

liggitt commented Apr 6, 2018

what worries me is that the current state is so tightly coupled with origin that it's impossible to build it and test it :/

not sure what you mean... I can run hack/local-up-cluster.sh and unit tests directly from this branch

Paul Weil and others added 17 commits April 7, 2018 10:46
:100644 100644 a9b6b37... 2328ffc... M	pkg/apiserver/api_installer.go
:100644 100644 c561ddb... 17ef5df... M	pkg/kubelet/server.go
…vider

Previously, if the kubelet tried to register itself with the API server,
and was rejected due to the external ID changing, it would delete the
node object and recreate it.  This commit causes it to tolerate
a change in ExternalID when the ExternalID is not being provided by a
cloud provider, assuming the new ExternalID is either the node's
(metadata) name, or one of node's addresses.

:100644 100644 0c4fdc1... 8deaa82... M	pkg/kubelet/kubelet.go
:100644 100644 b32534e... 3e694fc... M	pkg/controller/serviceaccount/tokens_controller.go
:100644 100644 eaa35e6... 2a50982... M	pkg/client/cache/fifo.go
:100644 100644 0f2ceb8... 667c168... M	pkg/client/cache/fifo_test.go
Doesn't offer enough use, complicates creation and setup.
:000000 100644 0000000000... b684356ed4... A	pkg/registry/core/rest/scc_patch.go
:100644 100644 66e2a7be20... ea1028e61e... M	pkg/registry/core/rest/storage_core.go
:000000 100644 0000000000... 748bed00fb... A	staging/src/k8s.io/apiserver/pkg/admission/patch.go
:100644 100644 dd1368d4dd... 0e422f990c... M	staging/src/k8s.io/apiserver/pkg/admission/plugins.go
:100644 100644 9fa52a0021... dab9e7233d... M	staging/src/k8s.io/apiserver/pkg/server/config.go
:100644 100644 9be725d4e2... 65e2ed1670... M	staging/src/k8s.io/apiserver/pkg/server/genericapiserver.go
:100644 100644 08e342ef56... 9218a1e185... M	staging/src/k8s.io/apiserver/pkg/server/routes/swagger.go
…nfig

:000000 100644 0000000000... 29aa4dd35d... A	cmd/kube-proxy/app/server_patch.go
deads2k and others added 27 commits April 7, 2018 10:50
:100644 100644 bf354932a8... d53f9c38a7... M	pkg/kubectl/cmd/util/factory_client_access.go
The legacy oapi v1 group-version very much confuses anything not
designed to explicitly work with it.  Since we now don't do any custom
HPA setup, we need to teach the scale client and the HPA what to do
with the oapi version of DC, since it won't even show up in its
discovery process.
…m to allow multiple containers to union for swagger
Drop once controllers are never run in-process with the API server (once `openshift start master` is removed)
… separate process

:100644 100644 e0c41b82f8... cb7771a9d4... M	pkg/controller/serviceaccount/serviceaccounts_controller.go
:100644 100644 a9bbe08a1f... ff1687114b... M	staging/src/k8s.io/kube-aggregator/pkg/controllers/status/available_controller.go
We don't run system pods so waiting for them is somewhat pointless
Until kubernetes#11016 is resolved, we suspect that a combination of start latency
and the corresponding effect on sync pod latency is causing status
manager to fail to report within the 2 minute window. Double for now

:100644 100644 30b6fd0e94... ea322bbddb... M	test/e2e/common/container_probe.go
Until kubernetes#11016 is fixed, this reduces flakiness in extended suites where
long start delays result in this test failing.

:100644 100644 62b8918f36... 99d37eca2f... M	test/e2e/framework/util.go
:100644 100644 31228219fe... 72987e330d... M	pkg/controller/node/timed_workers_test.go
Scheduling tests are too aggressive about the health of kube-system.

:100644 100644 b6ef25dc75... 4192ffbea8... M	test/e2e/scheduling/predicates.go
:100644 100644 6dab988e52... 74b9cce53a... M	staging/src/k8s.io/apiserver/pkg/server/config.go
:100644 100644 dd9ab0dcfe... fcaec9e519... M	staging/src/k8s.io/apiserver/pkg/server/routes/version.go
:100644 100644 4e0ce16e85... 68d79a7878... M	test/e2e/common/downwardapi_volume.go
@liggitt liggitt merged commit b81c8f8 into openshift:origin-3.10-kubernetes-1.10.0 Apr 7, 2018
@liggitt liggitt deleted the rebase-1.10.0 branch April 22, 2019 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants