adutra commented on issue #865: URL: https://github.com/apache/polaris/issues/865#issuecomment-2612059626
Hi @MonkeyCanCode @flyrain @HonahX sorry for creating so much confusion here. Here is my long term vision: - I think the test profile (used in automated tests) is too different from the prod profile. It is always good to make those converge as much as possible. That's why I opened #786. I think it would be good to merge that one quickly, I will do it today. - I also think that the regression tests should strive to use the same config as in production. This is not the case today, as you noticed, which is why I think you opened this issue. I started a PR (in draft) here: #804 where the authenticator for regression tests is changed to use the production one. - As much as possible, I think that the `test` authenticator + token broker should only be used for prototyping, quickstart guides and demos, but not for tests. Does that make sense? If that's OK with you, let's proceed as follows: - Get #786 merged - Get #804 reviewed and merged - Revisit the docs and fix all the places where the regression tests are not launched with the appropriate parameters. -- 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]
