Skip to content

Comments

Revert the Python 3.11 exclusion for date isoformat#32490

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:fix-back-py311-date-validation
Jul 10, 2023
Merged

Revert the Python 3.11 exclusion for date isoformat#32490
potiuk merged 1 commit intoapache:mainfrom
potiuk:fix-back-py311-date-validation

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jul 10, 2023

The official Python image for 3.11 had a brief period when the date 20120503 has been an acceptable date, but it is not accepted as of 3.11.4 release on 5th of July.

This PR reverts the changes in tests implemented in #27264 to accomodate for it and brings back the date as error case.


^ 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.

@potiuk potiuk requested a review from uranusjr July 10, 2023 12:11
Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good find. Looks good to me +1

@potiuk potiuk added the full tests needed We need to run full set of tests for this PR to merge label Jul 10, 2023
Copy link
Member

@hussein-awala hussein-awala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will conflict with #32445, I will rebase once this one is merged

The official Python image for 3.11 had a brief period when the
date `20120503` has been an acceptable date, but it is not
accepted as of 3.11.4 release on 5th of July.

This PR reverts  the changes in tests implemented in apache#27264 to
accomodate for it and brings back the date as error case.
@potiuk potiuk force-pushed the fix-back-py311-date-validation branch from 77b640e to cb8f643 Compare July 10, 2023 12:16
@potiuk
Copy link
Member Author

potiuk commented Jul 10, 2023

I am totally puzzled by this one BTW. I have no idea why the test worked with the previous base images :)

@hussein-awala
Copy link
Member

This is related to jsonschema validation, this will be fixed once we bump its version

@potiuk
Copy link
Member Author

potiuk commented Jul 10, 2023

Ah OK. Understood. Merging and yours right after.

@potiuk potiuk merged commit 60eccf0 into apache:main Jul 10, 2023
@potiuk potiuk deleted the fix-back-py311-date-validation branch July 10, 2023 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants