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

Enable default export of process-message-subscription record type #9370

Closed
sdorokhova opened this issue May 13, 2022 · 10 comments
Closed

Enable default export of process-message-subscription record type #9370

sdorokhova opened this issue May 13, 2022 · 10 comments
Assignees
Labels
kind/toil Categorizes an issue or PR as general maintenance, i.e. cleanup, refactoring, etc.

Comments

@sdorokhova
Copy link
Contributor

Description

Operate starts to import data from index process-message-subscription (related issue https://github.com/camunda-cloud/operate/issues/2750), therefore we need to have export for this index enabled by default.

@sdorokhova sdorokhova added the kind/toil Categorizes an issue or PR as general maintenance, i.e. cleanup, refactoring, etc. label May 13, 2022
@sdorokhova sdorokhova self-assigned this May 13, 2022
@Zelldon
Copy link
Member

Zelldon commented May 13, 2022

I think this will be done via #8338

@sdorokhova could you please clarify whether you want to see events only or are you also interested in commands?

@sdorokhova
Copy link
Contributor Author

sdorokhova commented May 13, 2022

Currently we only need CORRELATED events from process-message-subscription index.
And in general I would say that we import only events, not commands.

Do you know when the issue #8338 will be addressed? We need it to be implemented for https://github.com/camunda-cloud/operate/issues/2750.

@Zelldon
Copy link
Member

Zelldon commented May 13, 2022

I guess @remcowesterhoud @korthout can tell you :)

@sdorokhova
Copy link
Contributor Author

Thanks, @Zelldon !

As an idea. May be we can decouple these two tasks: enable one additional index for Operate purposes and enabling them all within #8338 ?

@remcowesterhoud
Copy link
Contributor

@sdorokhova Is this severely blocking you? I was going to look at it next week, which would not be earlier than Thursday because I'll be on holiday. If it's blocking you someone else should take over for me.

@sdorokhova
Copy link
Contributor Author

@remcowesterhoud No, this is not really blocking in case it's included in 8.1.0. Would be nice to have in alpha2, but not necessary.

@remcowesterhoud
Copy link
Contributor

👍 That should be fine then.

@sdorokhova
Copy link
Contributor Author

I gave it a second thought:)
In case it's not in alpha2, we would need to adjust (at least temporary) Zeebe configuration in Camunda Cloud. So I would need to know for sure, whether it's included in alpha2 or not. Could you give me such information?

@sdorokhova
Copy link
Contributor Author

@remcowesterhoud and another question. Should we close this issue then?

@remcowesterhoud
Copy link
Contributor

I'll be able to get it done before alpha2 so it'll be included. Yes, I'll close this issue and progress can be tracked in #8338

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/toil Categorizes an issue or PR as general maintenance, i.e. cleanup, refactoring, etc.
Projects
None yet
Development

No branches or pull requests

3 participants