You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Templates owned by Project Account not allowed to share templates #7350
publicbooleanupdateTemplateOrIsoPermissions(BaseUpdateTemplateOrIsoPermissionsCmdcmd) {
<...>
if (owner.getType() == Account.Type.PROJECT) {
// Currently project owned templates cannot be shared outside project but is available to all users within project by default.thrownewInvalidParameterValueException("Update template permissions is an invalid operation on template " + template.getName() +
". Project owned templates cannot be shared outside template.");
}
<...>
SUMMARY
Templates owned by a project account are not allowed to exectute updateTemplatePermissions API.
This prevents projects from sharing templates between them selves that are in the same Domain as stated by the API docs.
It seems to indicate that only templates created by a users in the "admin" or other non-project group are allowed to mark templates shared outside of the project structure.
EXPECTED RESULTS
Allowing users in the same domain to share templates would seem to be an expected behavior.
@weizhouapache@rohityadavcloud@harikrishna-patnala does it make sense to allow ROOT admin to change the owner of the templates using updateTemplate API?
Or should we allow project-owned templates to be shared outside the project?
This is by design, template registered in a project are owned by it so not accessible to others. However, like any account they should be allowed to be shared to others.
I added a simple change in #9196 . A domain admin can share with the PR at least. Normal users never own their own template in a project as it belongs to the project account. I am not sure if this is as intended, but it seems some extra work is needed.
@kiranchavala@rohityadavcloud@damonb123 , My take on this is that if the user has access to the template thay can adjust the sharing rights as well. I do not have clear yet if there should be some restrictions to what they can do, beyond what is programmatically possible at this moment. I.E. are restrictions and extra checks needed?
@damonb123#9196 implements your request . as @vishesh92 shows in #9196 (comment) this is not much help yet for UI users but may help you if you want to share for use in the API. I will look at a furhter UI fix.
Activity
kiranchavala commentedon Mar 20, 2023
Hi @damonb123
Could you please let me know what's role typer of the account
shwstppr commentedon Apr 6, 2023
@weizhouapache @rohityadavcloud @harikrishna-patnala does it make sense to allow ROOT admin to change the owner of the templates using updateTemplate API?
Or should we allow project-owned templates to be shared outside the project?
yadvr commentedon Apr 30, 2024
This is by design, template registered in a project are owned by it so not accessible to others. However, like any account they should be allowed to be shared to others.
DaanHoogland commentedon Jun 11, 2024
I added a simple change in #9196 . A domain admin can share with the PR at least. Normal users never own their own template in a project as it belongs to the project account. I am not sure if this is as intended, but it seems some extra work is needed.
DaanHoogland commentedon Jun 14, 2024
@kiranchavala @rohityadavcloud @damonb123 , My take on this is that if the user has access to the template thay can adjust the sharing rights as well. I do not have clear yet if there should be some restrictions to what they can do, beyond what is programmatically possible at this moment. I.E. are restrictions and extra checks needed?
DaanHoogland commentedon Jun 19, 2024
@damonb123 #9196 implements your request . as @vishesh92 shows in #9196 (comment) this is not much help yet for UI users but may help you if you want to share for use in the API. I will look at a furhter UI fix.
DaanHoogland commentedon Jun 21, 2024
fixed in #9196