Re: [PR] feat: add read access to pools for viewer role [airflow]

eladkal merged PR #35352: URL: https://github.com/apache/airflow/pull/35352 -- 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:

Re: [PR] feat: add read access to pools for viewer role [airflow]

potiuk commented on PR #35352: URL: https://github.com/apache/airflow/pull/35352#issuecomment-1834711150 > I'll wait for another approval before merging this. I just want to be sure that there is no historical reason for having it this way in the first place. You get it :) -- This

Re: [PR] feat: add read access to pools for viewer role [airflow]

raphaelauv commented on PR #35352: URL: https://github.com/apache/airflow/pull/35352#issuecomment-1831705291 the test `test_get_user_roles_for_anonymous_user` is failing -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] feat: add read access to pools for viewer role [airflow]

raphaelauv commented on PR #35352: URL: https://github.com/apache/airflow/pull/35352#issuecomment-1829723850 @marekhanus could you please rebase on main , to trigger a new CI test run -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] feat: add read access to pools for viewer role [airflow]

marekhanus commented on PR #35352: URL: https://github.com/apache/airflow/pull/35352#issuecomment-1806267400 Rebase done. Conflicts after https://github.com/apache/airflow/commit/ad717ef50d58bfc442f62a7da03e2a77acd636b9 are now resolved. -- This is an automated message from the Apache

Re: [PR] feat: add read access to pools for viewer role [airflow]

raphaelauv commented on PR #35352: URL: https://github.com/apache/airflow/pull/35352#issuecomment-1794877998 @marekhanus if your PR is ready you could set it in "ready for review" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] feat: add read access to pools for viewer role [airflow]

boring-cyborg[bot] commented on PR #35352: URL: https://github.com/apache/airflow/pull/35352#issuecomment-1789254629 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution

[PR] feat: add read access to pools for viewer role [airflow]

marekhanus opened a new pull request, #35352: URL: https://github.com/apache/airflow/pull/35352 Cluster Activity UI shows `FAILED TO FETCH DATA` error in pools section. Users with Viewer role do not have permission to read pools. --- **^ Add meaningful