Skip to content

Fix wrong test after fixing audit log permission#37547

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-missing-test-audit-logs
Feb 19, 2024
Merged

Fix wrong test after fixing audit log permission#37547
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-missing-test-audit-logs

Conversation

@potiuk

@potiuk potiuk commented Feb 19, 2024

Copy link
Copy Markdown
Member

The test for security/permissions had not been run after modifying default permissions in #37501 (to be investigated why). This PR makes main green again.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

The test for security/permissions had not been run after modifying
default permissions in apache#37501 (to be investigated why). This PR
makes main green again.
@boring-cyborg boring-cyborg Bot added the area:webserver Webserver related Issues label Feb 19, 2024
@potiuk potiuk merged commit e92b82e into apache:main Feb 19, 2024
@potiuk potiuk deleted the fix-missing-test-audit-logs branch February 19, 2024 15:31
@potiuk potiuk added this to the Airflow 2.8.2 milestone Feb 19, 2024
potiuk added a commit that referenced this pull request Feb 19, 2024
The test for security/permissions had not been run after modifying
default permissions in #37501 (to be investigated why). This PR
makes main green again.
ephraimbuddy pushed a commit that referenced this pull request Feb 22, 2024
The test for security/permissions had not been run after modifying
default permissions in #37501 (to be investigated why). This PR
makes main green again.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:webserver Webserver related Issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants