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

Spot clean try_number references to use ti.try_number #42633

Closed
wants to merge 1 commit into from

Conversation

dheerajturaga
Copy link

Some tasks in airflow 2.10.2 are being launched with try number 0. However, the webserver is looking for logs starting with try number 1. Upon further investigation, I suspect that {try_number} and {ti.try_number} are diverging leading to this.

This is an attempt to cleanup references to {try_number}

related: #42549, #39336

@dstandish
Copy link
Contributor

These two values should be the same. Do you have repro steps to demonstrate that they differ? Or perhaps a test?

@dheerajturaga
Copy link
Author

These two values should be the same. Do you have repro steps to demonstrate that they differ? Or perhaps a test?

Let me see if I can catch this somehow.

Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Nov 16, 2024
@dstandish
Copy link
Contributor

dstandish commented Nov 16, 2024

Author noted in #42549 (comment) that this is a false alarm.

@dstandish dstandish closed this Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Stale PRs per the .github/workflows/stale.yml policy file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants