Skip to content

Conversation

prashantmital
Copy link
Contributor

No description provided.

@@ -1137,6 +1137,42 @@ def test_versioned_api(self):
uri, server_api=ServerApi("1", deprecation_errors=True))
# End Versioned API Example 4

@client_context.require_version_min(4, 7)
Copy link
Member

Choose a reason for hiding this comment

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

Do we still need to skip sharded clusters?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The issue has been fixed in 5.0.2 - https://jira.mongodb.org/browse/SERVER-58794 but seems like we are still using 5.0. 🤔
Should I just skip it on sharded or we can wait for this to pass before merging?

Copy link
Member

Choose a reason for hiding this comment

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

Do you mind updating drivers evergreen tools to use 5.0.2 real quick?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

How does one figure out if it is available yet? I only see 5.0.1 on m...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah its definitely not yet available.

Copy link
Member

Choose a reason for hiding this comment

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

It's not out yet: https://docs.mongodb.com/manual/release-notes/5.0/

I suggest putting something like :

if sharded and 5.0 and <5.0.2:
    self.skip()


# Start Versioned API Example 8
# 8
# End Versioned API Example 7

Choose a reason for hiding this comment

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

Drive-by fix. I think this typo is also in motor here.

Suggested change
# End Versioned API Example 7
# End Versioned API Example 8

@prashantmital prashantmital merged commit f86b2c6 into mongodb:master Jul 30, 2021
@prashantmital prashantmital deleted the PYTHON-2827/versioned-api-migration-example branch July 30, 2021 00:17
prashantmital added a commit that referenced this pull request Jul 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants