ClusterRole and ClusterRoleBinding should be NonNamespacedOperation in RbacAPIGroupClient

[rohanKanojia]

ClusterRole and ClusterRoleBinding are Cluster scoped Kubernetes resources:

 $ kubectl api-resources
NAME                                  SHORTNAMES       APIVERSION                                    NAMESPACED   KIND

clusterrolebindings                                    authorization.openshift.io/v1                 false        ClusterRoleBinding
clusterroles                                           authorization.openshift.io/v1                 false        ClusterRole

But we're right now using MixedOperation for these which also exposes inNamespace method. It doesn't make sense to have this method for a Cluster Scoped resource. We should use NonNamespacedOperation instead.

RbacApiGroupDSL:

kubernetes-client/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/dsl/RbacAPIGroupDSL.java

Lines 34 to 37 in cc4945f

	MixedOperation<ClusterRole, ClusterRoleList, Resource<ClusterRole>> clusterRoles();
	MixedOperation<ClusterRoleBinding, ClusterRoleBindingList, Resource<ClusterRoleBinding>> clusterRoleBindings();

RbacApiGroupClient

kubernetes-client/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/RbacAPIGroupClient.java

Lines 48 to 57 in cc4945f

	@Override
	public MixedOperation<ClusterRole, ClusterRoleList, Resource<ClusterRole>> clusterRoles() {
	return new ClusterRoleOperationsImpl(httpClient, getConfiguration());
	}
	@Override
	public MixedOperation<ClusterRoleBinding, ClusterRoleBindingList, Resource<ClusterRoleBinding>> clusterRoleBindings() {
	return new ClusterRoleBindingOperationsImpl(httpClient, getConfiguration());
	}
	}

We need to modify clusterRoles() and clusterRoleBindings() to return NonNamespaceOperation instead of MixedOperation

[MUzairS15]

I would like to try.

[rohanKanojia]

@MUzairS15 : Shall I assign issue to you then?

[sourabhkumar45]

ready to work on this issue, Assign if possible.

[MUzairS15]

yes sure

[MUzairS15]

How do i check if i had been successfull, running which test can help?

[rohanKanojia]

@MUzairS15 : Are you working on this issue? Running mvn clean install in kubernetes-client/ module might help. Or running mvn clean install -DskipTests on top of the module might help too to make sure you haven't introduced any compilation failures. We have mock tests in kubernetes-tests/ module which you can run. We have also got tests in kubernetes-itests/ module which you can run against a live kubernetes cluster

[rohanKanojia]

@sourabhkumar45 : I've created two more good first issues. Could you please check if you can pick any one of these instead?

#3162
#3163