claw89 commented on PR #43446: URL: https://github.com/apache/airflow/pull/43446#issuecomment-2454824832
> > > Given the operators are for allowlist-only service, I don't think they should be placed in the main Google provider, but instead in a separate one. > > > > > > Thanks for the suggestion. Would you recommend adding to somewhere existing in the provider packages or introducing a new provider? > > I think it should be a new, separate one. Ideally, providers should come with dashboards with system tests, similar to the ones we already have: > > * Astronomer (found this LLM-specific one, not sure if there is another one): https://astronomer.github.io/llm-dags-dashboard/ > * AWS: https://aws-mwaa.github.io/open-source/system-tests/dashboard.html and https://aws-mwaa.github.io/open-source/system-tests/dashboard.json > * Google: https://storage.googleapis.com/providers-dashboard-html/dashboard.html Can I clarify that your concern about the system test is access to the Financial Services API? I assume that the API is not currently available in the system test environment. Would it be possible to enable the API in the system test environment, and would that address your concerns? -- 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]
