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

Add support for TenantId #591

Open
ShawnAbshire opened this issue Nov 1, 2023 · 7 comments · May be fixed by #694
Open

Add support for TenantId #591

ShawnAbshire opened this issue Nov 1, 2023 · 7 comments · May be fixed by #694
Assignees
Labels

Comments

@ShawnAbshire
Copy link
Contributor

ShawnAbshire commented Nov 1, 2023

Describe the solution you'd like
Adding a new argument to to the various builders to add a tenantId. New commands that support tenantId look to be:

NewActivateJobsCommand()
NewCreateProcessInstanceCommand()
NewDeployCommand()
NewEvaluateDecisionCommand()
NewPublishMessageCommand()

I imagine something like the following would be ideal:

IDeployResourceResponse? deployResponse = await client
	.NewDeployCommand()
        .AddTenantId("12345")
	.AddResource("test.bpmn")
	.Send();

Additional context
With the release of Camunda 8.3, it added support for multi-tenancy.

@ShawnAbshire
Copy link
Contributor Author

ShawnAbshire commented Nov 2, 2023

Hey @Zelldon - Got a thumbs up from ya, but wanted to follow up and see if this is an active feature you're working on or if it's something you'd prefer the community to add? This a feature I'm highly looking forward to being able to leverage as soon as possible so just wanted to gauge the timeline.

Thanks!

@Zelldon
Copy link
Collaborator

Zelldon commented Nov 2, 2023

Thumbs up for raising interest in the project and a missing feature :)

I will not give any eta since it is a side project I working on in my free time but sure go forward and create a PR we can discuss details their.

@Zelldon
Copy link
Collaborator

Zelldon commented Nov 2, 2023

Would be great if you could do this in small chunks one per command incl tests

@ShawnAbshire
Copy link
Contributor Author

Got a little busy with other projects and deadlines, but intend to revisit this feature request fairly soon as I believe multi tenancy support for camunda 8 cloud is slated for an April release and we will need to take advantage of it asap.

Thanks for your patience!

@hmeniz
Copy link

hmeniz commented Apr 15, 2024

Hi,

Is there a date for adding the tenantId feature when creating a worker like in java client?

client.newWorker()
.jobType("myJobType")
.handler(new MyJobTypeHandler())
.tenantId("myTenant")
.open();

client.newWorker()
.jobType("myJobType")
.handler(new MyJobTypeHandler())
.tenantIds("myTenant", "myOtherTenant")
.open();

@ShawnAbshire
Copy link
Contributor Author

No set date for me completing this work. Multi-Tenancy support for Camunda 8 cloud was originally suppose to hit in April but has been delayed until later this year and I'm tied up with other things. Believe the foundation has been set for updating the remaining interfaces though.

@ShawnAbshire
Copy link
Contributor Author

ShawnAbshire commented Sep 23, 2024

Looks like multi-tenancy support in the camunda saas has been delayed by another release.... hopefully we get it in Janurary.. In the meantime any word on the final 2 PRs would be appreciated so this issue can be closed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants