blankensteiner commented on pull request #95: URL: https://github.com/apache/pulsar-dotpulsar/pull/95#issuecomment-1028730259
We might need multiple instances in the future, but the old tests could just have a token added and then we only have to maintain one yml-file and only have to spin up and wait for one instance. I also think that, at some point, it would be nice to merge "StressTests" and "IntegrationTests" into one project. The token didn't work for me locally, but it worked when creating a new token. Now the token doesn't work in the workflow. I would have loved that Pulsar made it easy to "one line" an entire test setup with TLS and Authentication and so on. Not sure how to solve this as the Metadata exception we are seeing is because the token the broker is using doesn't validate for some weird reason. If you prefer I can guide you more on the next PR, instead of just merging and changing stuff myself. I did it because experience tells me that to much guidance can make some abandon the PR. -- 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: dev-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
