tenthe commented on PR #3958: URL: https://github.com/apache/streampipes/pull/3958#issuecomment-3573937146
Hi @dominikriemer, I really like this change, as it simplifies the user flow. I have a question regarding existing charts: How does it affect existing charts? Do we need a migration for them? I’m currently working on the permissions. This change also allows us to remove the permissions for pipleline reads for chart users and chart admins, which helps eliminate the dependency on pipelines. Regarding the E2E tests, I suggest running the entire test suite on this branch as well, since there may be breaking changes that wouldn’t be caught by the smoke tests alone. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
