Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create latest log dir symlink as relative link #36019

Merged
merged 1 commit into from
Dec 9, 2023

Conversation

Kache
Copy link
Contributor

@Kache Kache commented Dec 1, 2023

So logs/**/* is self-contained and the symlink doesn't break for
exporting, build artifact, etc

So `logs/**/*` is self-contained and the symlink doesn't break for
exporting, build artifact, etc
@Kache
Copy link
Contributor Author

Kache commented Dec 1, 2023

Rebased from: #36016

@eladkal eladkal requested a review from potiuk December 9, 2023 09:03
@potiuk potiuk merged commit 17e9172 into apache:main Dec 9, 2023
47 checks passed
@potiuk potiuk added this to the Airflow 2.8.0 milestone Dec 9, 2023
@eladkal
Copy link
Contributor

eladkal commented Dec 9, 2023

window for 2.8.0 is closed will have to wait for next release

@eladkal eladkal modified the milestones: Airflow 2.8.0, Airflow 2.8.1 Dec 9, 2023
@potiuk
Copy link
Member

potiuk commented Dec 9, 2023

Ech... yeah .... sneaking things in the last moment :)

@ephraimbuddy ephraimbuddy added the type:bug-fix Changelog: Bug Fixes label Jan 10, 2024
ephraimbuddy pushed a commit that referenced this pull request Jan 11, 2024
So `logs/**/*` is self-contained and the symlink doesn't break for
exporting, build artifact, etc

(cherry picked from commit 17e9172)
@Kache Kache deleted the kache/relative-latest-log-symlink branch February 6, 2024 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:logging type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants