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

feat: add Operate and Tasklist issues/sections #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

houssain-barouni
Copy link

Add Tasklist and Operate sections in the changelog with respective Tasklist and Operate issues labeled with component/tasklist and component/operate

@ChrisKujawa
Copy link
Member

I was wondering whether we maybe should fork this repo and create it in the camunda org, and then adjust it to our needs. As right now it is intended to work with Zeebe and also their patch releases, but we want to get it work with the mono-repo and soon called Camunda application. For me it might make sense to have a cleaner cut here. Wdyt?

@npepinpe @romansmirnov any opinion?

@npepinpe
Copy link
Member

I would prefer not having to build such a tool in the first place, especially if we're migrating all teams to it 🤔

@ChrisKujawa
Copy link
Member

ChrisKujawa commented May 14, 2024

What would be the alternative :D I guess generating changelog by Github? But somewhere you need some glue code to make it "pretty" and adjust it to your needs. Either with configuration and using some tools/actions or simply doing like done with zcl 🤷🏼

@houssain-barouni
Copy link
Author

Hi @npepinpe, can we have this merged if it is ok ? That will help generating Tasklist and Operate release notes that is missing today in the mono-repo.

Copy link
Member

@npepinpe npepinpe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine, note that I did not test it, I assume you already did :)

I still maintain we should drop this tool 😄

@houssain-barouni
Copy link
Author

@npepinpe @Zelldon
I tested with the following

export ZCL_FROM_REV=8.6.0-alpha1
export ZCL_TARGET_REV=8.6.0-alpha2

I tested first adding the version: 8.6.0-alpha2 label to the issues

  zcl add-labels \
    --token=$GITHUB_TOKEN \
    --from=$ZCL_FROM_REV \
    --target=$ZCL_TARGET_REV \
    --label="version:$ZCL_TARGET_REV" \
    --org camunda --repo zeebe

but got fatal: bad revision '8.6.0-alpha1..8.6.0-alpha2', I think it is due to that 8.6.0-alpha1 tag does not belong to any branch as release-8.6.0-alpha1 was deleted without merging to main

Then tested the generation of ChangeLog with the existing labeled issues

  ./bin/zcl generate \
     --token=$GITHUB_TOKEN \
     --label="version:$ZCL_TARGET_REV" \
     --org camunda --repo camunda

and it is working including Tasklist and Operate sections

2024/06/14 14:14:47 Fetching issues for GitHub label version:8.6.0-alpha2
2024/06/14 14:14:48 Generating changelog for GitHub label version:8.6.0-alpha2
# version:8.6.0-alpha2
## Enhancements
### Broker
* Migrate child process instances ([#15922](https://github.com/camunda/camunda/issues/15922))
* Migrate call activity instances ([#15921](https://github.com/camunda/camunda/issues/15921))
### Operate
* Pre-select target proceess versions in instance migration mode ([#18761](https://github.com/camunda/camunda/issues/18761))
* I can filter for unmapped source flow nodes in instance migration mode ([#18573](https://github.com/camunda/camunda/issues/18573))
* I can see which flow nodes are not mapped in instance migration mode ([#18571](https://github.com/camunda/camunda/issues/18571))
* I can see automapping for flow nodes in instance migration mode ([#18538](https://github.com/camunda/camunda/issues/18538))
* Support call activity mapping in instance migration mode ([#18492](https://github.com/camunda/camunda/issues/18492))
### Misc
* Review and address TODOs ([#17342](https://github.com/camunda/camunda/issues/17342))
* As a user I want to configure `LongPollingActivateJobsHandler` ([#8777](https://github.com/camunda/camunda/issues/8777))
## Bug Fixes
### Gateway
* Endless errors on gateway because the writer is full ([#13018](https://github.com/camunda/camunda/issues/13018))
### Operate
* Operate import does not work for Opensearch with `UnsupportedOperationException` ([#18452](https://github.com/camunda/camunda/issues/18452))
### Tasklist
* Tasklist cannot update ISM policy in AWS OpenSearch ([#18891](https://github.com/camunda/camunda/issues/18891))
* Missing Zeebe user task migration doc URL in REST API error message ([#18663](https://github.com/camunda/camunda/issues/18663))
### Misc
* Remove client's secrets reference in logs ([#18656](https://github.com/camunda/camunda/issues/18656))
* When activating the identity-auth profile, the health and readiness probes fail with status code 401 ([#18568](https://github.com/camunda/camunda/issues/18568))
## Maintenance
* Run Spotbugs plugin on the Operate codebase and fix related issues ([#17331](https://github.com/camunda/camunda/issues/17331))
* Extend generalized command distribution for subsets of partitions ([#16385](https://github.com/camunda/camunda/issues/16385))
* Dead code in metrics exporter ([#9760](https://github.com/camunda/camunda/issues/9760))
## Merged Pull Requests
* fix: Tasklist cannot update ISM policy in AWS OpenSearch ([#18924](https://github.com/camunda/camunda/pull/18924))
* Generate changelog for patch releases ([#18342](https://github.com/camunda/camunda/pull/18342))
* Migrate active call activity ([#18328](https://github.com/camunda/camunda/pull/18328))
* Observe status of Zeebe CI jobs ([#18224](https://github.com/camunda/camunda/pull/18224))
* fix(docs): fix links to config files in docs ([#4545](https://github.com/camunda/camunda/pull/4545))

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

Successfully merging this pull request may close these issues.

3 participants