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

Standard User can not create cluster #13483

Closed
gravityforall-zz opened this issue May 14, 2018 · 8 comments
Closed

Standard User can not create cluster #13483

gravityforall-zz opened this issue May 14, 2018 · 8 comments
Assignees
Labels
kind/bug Issues that are defects reported by users or that we know have reached a real release
Milestone

Comments

@gravityforall-zz
Copy link

gravityforall-zz commented May 14, 2018

Rancher versions:
CATTLE_UI_VERSION = 2.0.41
CATTLE_CLI_VERSION = v2.0.0
CATTLE_SERVER_VERSION = v2.0.0
CATTLE_AGENT_IMAGE = rancher/rancher-agent:v2.0.0

Docker version: (docker version,docker info preferred)
Docker version 17.03.2-ce, build f5ec1e2

Operating system and kernel: (cat /etc/os-release, uname -r preferred)
NAME="Ubuntu"
VERSION="16.04.4 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.4 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial
4.4.0-119-generic

Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Rackspace/OpenStack

Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher

Steps to Reproduce:

  1. Leave the 'Cluster Name' field blank when creating a new cluster

screen shot 2018-05-14 at 1 46 25 pm

Results:
Error 'User XXX cannot create node pools...'
screen shot 2018-05-14 at 1 46 46 pm

@tfiduccia
Copy link

@gravityforall - Ryan, this is a bad error message. This is because you don't have Cluster Name. Which is a required field.

@gravityforall-zz gravityforall-zz changed the title Attempting to create a cluster as a 'Standard User' results in strange behavior Misleading error message when no name is specified during cluster creation May 14, 2018
@loganhz loganhz self-assigned this May 14, 2018
@loganhz
Copy link

loganhz commented May 15, 2018

I can reproduce this issue in master 05/14

  1. create a Standard User user1
  2. login as this user user1
  3. create a RKE cluster with DO node driver
  4. Input cluster name and select other fields
  5. Click create. The request for creating nodepool gets 403 with the similar error message.

I think this is a backend issue. It looks like we can do nothing from UI side.

@loganhz loganhz removed their assignment May 15, 2018
@loganhz loganhz removed the area/ui label May 15, 2018
@loganhz loganhz changed the title Misleading error message when no name is specified during cluster creation Standard User can not create cluster May 21, 2018
@loganhz loganhz added this to the v2.0.2 milestone May 21, 2018
@deniseschannon
Copy link

@prachidamle i thought we made the cluster name required in #13075

@deniseschannon deniseschannon changed the title Standard User can not create cluster Standard User can not create cluster without cluster name Jun 9, 2018
@prachidamle
Copy link
Member

@deniseschannon @loganhz yes, clusterName is now a required field and has to be unique as well

@loganhz
Copy link

loganhz commented Jun 10, 2018

@deniseschannon @prachidamle Standard user still can not create cluster with master 06/10.
It's not a cluster name issue. Maybe we should rename the issue.

clusterregistrationtokens.management.cattle.io is forbidden: User "u-c9qrf" cannot create clusterregistrationtokens.management.cattle.io in the namespace "c-98wgj"

@loganhz
Copy link

loganhz commented Jun 10, 2018

If standard users create a custom or import cluster
POST https://localhost/v3/clusterregistrationtoken returns 403.

If standard users create a cluster with node driver

POST https://localhost/v3/nodepool returns 403

image

@loganhz loganhz changed the title Standard User can not create cluster without cluster name Standard User can not create cluster Jun 10, 2018
@loganhz
Copy link

loganhz commented Jun 10, 2018

Duplicated with #12532

@loganhz loganhz closed this as completed Jun 10, 2018
@prachidamle
Copy link
Member

prachidamle commented Jun 10, 2018

The error message has been fixed to indicate that clusterName is a required field, as seen below:

screen shot 2018-06-10 at 6 02 32 am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Issues that are defects reported by users or that we know have reached a real release
Projects
None yet
Development

No branches or pull requests

7 participants