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.

Sign up for GitHub

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

Jump to bottom

feat: operate exporter #19594

Draft
wants to merge 20 commits into
base: main
Choose a base branch
from
Draft

feat: operate exporter #19594

wants to merge 20 commits into from

Conversation

matt-whiteman
Copy link
Contributor

Description

Related issues

closes #

mihail-ca and others added 20 commits June 6, 2024 12:34
@mihail-ca
feat: Implement BatchRequest using new ES client
5583e00 
feat: Implement BatchRequest using new ES client

-

Closes #19108
@mihail-ca
feat: Implement BatchRequest using new ES client
e0c5083
@mihail-ca
feat: Implement BatchRequest using new ES client (#19111)
fff7f04 
feat: Implement BatchRequest using new ES client

Closes #19108
@matt-whiteman
feat: first naive implementation of Operate exporter
14ceadd
@sdorokhova @matt-whiteman
fix: checkstyle fixes
41138db
@sdorokhova @matt-whiteman
fix: fix dependency
f94c033
@matt-whiteman
fix: fix compile errors due to rebase
741fdd3
@matt-whiteman
fix: fix use of indexprefix
675769e
@matt-whiteman
refactor: inject xmlutil through constructor
4b144ec
@matt-whiteman
feat(operate): new exporter module (#18886)
7874780 
## Description

* new exporter logic to write data directly to Operate indices
* very simply structure to start from
* one sample data handler that converts and flushes process definitions
`ProcessHandler`

## Related issues

related to #18304
@matt-whiteman
feat: add variable handler to operate export
e0d196a
@matt-whiteman
Merge branch 'main' of https://github.com/camunda/camunda into feat-o…
44a610b 
…perate-exporter
@matt-whiteman
Merge branch 'feat-operate-exporter' of https://github.com/camunda/ca…
609c1a0 
…munda into 18915-add-exporter-modules
@matt-whiteman
fix: fix undeclared dependencies for operate exporter
e897007
@sdorokhova
fix: enable processing of variables
fbba034
@matt-whiteman
fix: set id in variable handler when creating new entity
10724e9
@matt-whiteman
feat: add variable handler to operate export (#19318)
9c0e64a 
## Description

- Add VariableHandler export handler
- Fix issue with tenant not being calculated correctly
- Add mockito/junit 5 to dependencies for unit tests

## Related issues

related #18915
@matt-whiteman
feat: add SequenceFlowHandler to operate exporter
3fc0fb7
@matt-whiteman
feat: add SequenceFlowHandler to operate exporter (#19394)
e4af052 
## Description

- Adds SequenceFlowHandler and unit test

## Related issues

related to #18915
@matt-whiteman
Merge branch 'main' of https://github.com/camunda/camunda into feat-o…
2211ce5 
…perate-exporter
@github-actions github-actions bot added component/zeebe Related to the Zeebe component/team component/operate Related to the Operate component/team component/tasklist Related to the Tasklist component/team labels Jun 20, 2024
@github-actions GitHub Actions
Copy link
Contributor

Operate Opensearch ITs Results

182 tests  ±0   182 ✅ ±0   53s ⏱️ -10s
 19 suites ±0     0 💤 ±0 
 19 files   ±0     0 ❌ ±0 

Results for commit 2211ce5. ± Comparison against base commit 078b5ba.

@sdorokhova sdorokhova added the hold/do-not-merge Used for work that needs further review and should not be merged label Jun 20, 2024
@github-actions GitHub Actions
Copy link
Contributor

Operate Unit Tests Results

308 tests  ±0   307 ✅ ±0   6m 43s ⏱️ +22s
 51 suites ±0     1 💤 ±0 
 51 files   ±0     0 ❌ ±0 

Results for commit 2211ce5. ± Comparison against base commit 078b5ba.

@github-actions GitHub Actions
Copy link
Contributor

Operate Integration Tests Results

 89 files  ±0   89 suites  ±0   9m 6s ⏱️ + 1m 3s
535 tests ±0  533 ✅ ±0  2 💤 ±0  0 ❌ ±0 
535 runs  +8  533 ✅ +8  2 💤 ±0  0 ❌ ±0 

Results for commit 2211ce5. ± Comparison against base commit 078b5ba.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 20, 2024 •
edited
Loading

Tasklist Test Results

564 tests  ±0   559 ✅ ±0   1h 35m 43s ⏱️ -47s
146 suites ±0     5 💤 ±0 
146 files   ±0     0 ❌ ±0 

Results for commit 2211ce5. ± Comparison against base commit 078b5ba.

This pull request removes 13 and adds 13 tests. Note that renamed tests count towards both. 
io.camunda.tasklist.webapp.api.rest.v1.controllers.external.ProcessExternalControllerTest ‑ startProcessWithEmbeddedForm(HttpStatus, String, ProcessEntity, ProcessInstanceDTO)[1] expectedHttpStatus=200 OK, bpmnProcessId=StartProcessByForm, processEntity=TenantAwareTasklistEntity[id='1', tenantId='<default>'], providedProcessInstanceDTO=io.camunda.tasklist.webapp.graphql.entity.ProcessInstanceDTO@6e359a1b
io.camunda.tasklist.webapp.api.rest.v1.controllers.external.ProcessExternalControllerTest ‑ startProcessWithLinkedForm(HttpStatus, String, ProcessEntity, ProcessInstanceDTO)[1] expectedHttpStatus=200 OK, bpmnProcessId=StartProcessByForm, processEntity=TenantAwareTasklistEntity[id='1', tenantId='<default>'], providedProcessInstanceDTO=io.camunda.tasklist.webapp.graphql.entity.ProcessInstanceDTO@f3d0efb5
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginAndUsageOfGraphQlApi(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x00007d1b3d2eef70@28a60d44
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginAndUsageOfRestApi(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x00007d1b3d2eef70@28a60d44
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginFailedWithNoPermissions(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x00007d1b3d2eef70@28a60d44
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginFailedWithNoReadPermissions(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x00007d1b3d2eef70@28a60d44
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginSucceedWithNoWritePermissions(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x00007d1b3d2eef70@28a60d44
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginSuccess(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x00007d1b3d2eef70@28a60d44
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLogout(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x00007d1b3d2eef70@28a60d44
io.camunda.tasklist.webapp.security.sso.AuthenticationWithPersistentSessionsIT ‑ testLoginFailedWithNoPermissions(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationWithPersistentSessionsIT$$Lambda/0x000079fd2931a820@4f18356b
…

io.camunda.tasklist.webapp.api.rest.v1.controllers.external.ProcessExternalControllerTest ‑ startProcessWithEmbeddedForm(HttpStatus, String, ProcessEntity, ProcessInstanceDTO)[1] expectedHttpStatus=200 OK, bpmnProcessId=StartProcessByForm, processEntity=TenantAwareTasklistEntity[id='1', tenantId='<default>'], providedProcessInstanceDTO=io.camunda.tasklist.webapp.graphql.entity.ProcessInstanceDTO@d970918a
io.camunda.tasklist.webapp.api.rest.v1.controllers.external.ProcessExternalControllerTest ‑ startProcessWithLinkedForm(HttpStatus, String, ProcessEntity, ProcessInstanceDTO)[1] expectedHttpStatus=200 OK, bpmnProcessId=StartProcessByForm, processEntity=TenantAwareTasklistEntity[id='1', tenantId='<default>'], providedProcessInstanceDTO=io.camunda.tasklist.webapp.graphql.entity.ProcessInstanceDTO@c8b2e0d5
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginAndUsageOfGraphQlApi(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x000078c13523bd78@2b0d0ef
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginAndUsageOfRestApi(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x000078c13523bd78@2b0d0ef
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginFailedWithNoPermissions(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x000078c13523bd78@2b0d0ef
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginFailedWithNoReadPermissions(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x000078c13523bd78@2b0d0ef
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginSucceedWithNoWritePermissions(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x000078c13523bd78@2b0d0ef
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLoginSuccess(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x000078c13523bd78@2b0d0ef
io.camunda.tasklist.webapp.security.sso.AuthenticationIT ‑ testLogout(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationIT$$Lambda/0x000078c13523bd78@2b0d0ef
io.camunda.tasklist.webapp.security.sso.AuthenticationWithPersistentSessionsIT ‑ testLoginFailedWithNoPermissions(BiFunction)[1] orgExtractor=io.camunda.tasklist.webapp.security.sso.AuthenticationWithPersistentSessionsIT$$Lambda/0x000079cd35318440@3445a757
…

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
component/operate Related to the Operate component/team component/tasklist Related to the Tasklist component/team component/zeebe Related to the Zeebe component/team hold/do-not-merge Used for work that needs further review and should not be merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@matt-whiteman @sdorokhova @mihail-ca