Skip to content

PYTHON-3110 [v3.13] Remove use of example.com in CSFLE tests (#848) #851

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

Merged
merged 3 commits into from
Feb 10, 2022

Conversation

blink1073
Copy link
Member

(cherry picked from commit 561ee7c)

@blink1073 blink1073 changed the title PYTHON-3110 Remove use of example.com in CSFLE tests (#848) PYTHON-3110 [v3.1.3] Remove use of example.com in CSFLE tests (#848) Feb 8, 2022
@blink1073
Copy link
Member Author

Scheduled encryption runs for Python 2.7 and 3.9

@blink1073 blink1073 changed the title PYTHON-3110 [v3.1.3] Remove use of example.com in CSFLE tests (#848) PYTHON-3110 [v3.13] Remove use of example.com in CSFLE tests (#848) Feb 8, 2022
}
with self.assertRaisesRegex(EncryptionError, 'parse error'):
with self.assertRaisesRegex(EncryptionError, self.invalid_host_error):
Copy link
Member

Choose a reason for hiding this comment

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

invalid_host_error hasn't been defined

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Member Author

Choose a reason for hiding this comment

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

And rescheduled encryption tasks

@blink1073 blink1073 requested a review from ShaneHarvey February 8, 2022 19:47
@blink1073
Copy link
Member Author

The Encryption Python 3.9 Amazon Linux 2018 (Enterprise) NoAuth NoSSL build is failing with sh: line 2: docker: command not found, I scheduled Encryption Python 3.8 Amazon Linux 2018 (Enterprise) NoAuth NoSSL.

@blink1073
Copy link
Member Author

Same issue with Encryption Python 3.8 Amazon Linux 2018 (Enterprise) NoAuth NoSSL, trying with a Windows variant.

@blink1073
Copy link
Member Author

Ah, I was misreading the logs. I fixed the missing re import and re-triggered Encryption runs for 2.7 and 3.9

@blink1073 blink1073 requested a review from juliusgeo February 10, 2022 00:13
Copy link
Contributor

@juliusgeo juliusgeo left a comment

Choose a reason for hiding this comment

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

LGTM!

@blink1073 blink1073 merged commit cc89650 into mongodb:v3.13 Feb 10, 2022
@blink1073 blink1073 deleted the PYTHON-3110-313 branch February 10, 2022 12:29
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