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

The org admin is able to create a job template using an execution environment from other organization #9518

Closed
yagomarques opened this issue Mar 8, 2021 · 4 comments

Comments

@yagomarques
Copy link
Contributor

ISSUE TYPE
  • Bug Report
SUMMARY
  • associate an execution_environment in a job template in which the Execution Environment is no longer global and has been linked to an other organization than the organization related to the job template.
STEPS TO REPRODUCE

reproducing by API

  1. create an organization A
  2. create an organization B
  3. create a global execution environment A
  4. create a project A related to organization A
  5. associate execution environment A with the organization B
  6. create an user with the org admin role and associate with the organization A
  7. create a job_template associating with project A and selecting execution_environment A.
  8. check that Job Template was associated an organization A and the project A. But the Job Template is using an execution environment associated an organization B
EXPECTED RESULTS
  • should to return an error, because the Execution Environment is no longer available to the organization
ACTUAL RESULTS
  • the job template was successfully registered
ADDITIONAL INFORMATION

we try to reproduce that behave with different users(sys_admin, org_admin and the ee_org_admin).

  • sys_admin - is able too to exec the same behave.
  • ee_org_admin - don't have access to create a new job_template.
@jbradberry
Copy link
Contributor

@yagomarques you should distinguish between the cases of 1) trying to create a JT with the EE already attached, and 2) creating a JT then after creation trying to attach the EE.

@masbahnana
Copy link
Contributor

masbahnana commented Mar 26, 2021

all tests are passing.

@masbahnana
Copy link
Contributor

masbahnana commented Mar 26, 2021

all tests are passing.

@masbahnana masbahnana reopened this Mar 26, 2021
@tiagodread
Copy link
Contributor

Looks like this was reopened accidentally with the comment

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

6 participants