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

Added SSL check to checkbox support (BugFix) #1254

Merged
merged 2 commits into from
May 22, 2024

Conversation

fernando79513
Copy link
Collaborator

@fernando79513 fernando79513 commented May 22, 2024

Description

Python 3.5 setup was failing because of a CERTIFICATE_VERIFY_FAILED error. To fix this, we have set up manually PIP_TRUSTED_HOST, checking first that we can "curl" the hosts, since they will fail in case of expired/invalid/self-signed certificate

Resolved issues

Documentation

N/A

Tests

The pipelines have been run with this changes and are all passing

Copy link

codecov bot commented May 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.51%. Comparing base (ba754d6) to head (c79a131).
Report is 9 commits behind head on main.

Current head c79a131 differs from pull request most recent head e015993

Please upload reports for the commit e015993 to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1254      +/-   ##
==========================================
+ Coverage   43.37%   43.51%   +0.13%     
==========================================
  Files         357      357              
  Lines       38693    38706      +13     
  Branches     6561     6564       +3     
==========================================
+ Hits        16784    16841      +57     
+ Misses      21245    21200      -45     
- Partials      664      665       +1     
Flag Coverage Δ
checkbox-ng 67.54% <ø> (ø)
checkbox-support 52.22% <ø> (ø)
provider-base 17.12% <ø> (+0.35%) ⬆️
provider-certification-client 57.14% <ø> (ø)
provider-certification-server 57.14% <ø> (ø)
provider-genio 96.90% <ø> (ø)
provider-gpgpu 57.14% <ø> (ø)
provider-iiotg 100.00% <ø> (ø)
provider-resource 28.63% <ø> (ø)
provider-sru 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fernando79513 fernando79513 force-pushed the fix-python35-pip-certificate-issue branch 3 times, most recently from f87e00e to 1fe2703 Compare May 22, 2024 14:21
@fernando79513 fernando79513 requested a review from a team as a code owner May 22, 2024 14:21
@fernando79513 fernando79513 force-pushed the fix-python35-pip-certificate-issue branch from 1fe2703 to 1b01e6e Compare May 22, 2024 14:27
@fernando79513 fernando79513 force-pushed the fix-python35-pip-certificate-issue branch from c79a131 to e015993 Compare May 22, 2024 14:45
Copy link
Collaborator

@Hook25 Hook25 left a comment

Choose a reason for hiding this comment

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

ty for working on this, looks very good, +1

@Hook25 Hook25 merged commit 1fbb8c4 into main May 22, 2024
6 checks passed
@Hook25 Hook25 deleted the fix-python35-pip-certificate-issue branch May 22, 2024 14:50
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.

2 participants