Skip to content

Update cryptography package#1577

Merged
kavitha-f5 merged 1 commit intoF5Networks:developmentfrom
sravyap135:crypto
May 10, 2021
Merged

Update cryptography package#1577
kavitha-f5 merged 1 commit intoF5Networks:developmentfrom
sravyap135:crypto

Conversation

@sravyap135
Copy link
Copy Markdown
Contributor

No description provided.

@zhaoqin-github
Copy link
Copy Markdown
Contributor

@sravyap135 Would you please restart your check?

@sravyap135
Copy link
Copy Markdown
Contributor Author

@zhaoqin-github, Observing error when trying to restart : An error occurred. The job could not be restarted.
image

@zhaoqin-github
Copy link
Copy Markdown
Contributor

@sravyap135 I am not sure what is wrong in Travis... You may use git commit --amend to generate a new commit id and push it again. Travis should trigger a new build.

@zhaoqin-github
Copy link
Copy Markdown
Contributor

@sravyap135 The failure in your latest check should be fixed by bfd2bac

Let me merge it first, and your check should be fine after a rebase.

@zhaoqin-github
Copy link
Copy Markdown
Contributor

@sravyap135 The ci in development branch is OK now. Please rebase your branch. Thank you!

@sravyap135
Copy link
Copy Markdown
Contributor Author

@zhaoqin-github Job succeeds for python 2.7, 3.5 and fails for python 3.6 with error could not install deps [-rrequirements.test.txt, git+https://github.com/F5Networks/f5-icontrol-rest-python.git@1.0]

@zhaoqin-github
Copy link
Copy Markdown
Contributor

@zhaoqin-github Job succeeds for python 2.7, 3.5 and fails for python 3.6 with error could not install deps [-rrequirements.test.txt, git+https://github.com/F5Networks/f5-icontrol-rest-python.git@1.0]

I am not quite sure, but this error is listed here https://cryptography.io/en/latest/faq/#installing-cryptography-fails-with-error-can-not-find-rust-compiler

The CI build in development branch can pass without you code change. That indicates your code change might not be compatible with every python version. Maybe you can switch to another version of cryptography, for example version 3.3.2, and have a try.

@sravyap135
Copy link
Copy Markdown
Contributor Author

Thanks @zhaoqin-github, It worked !

@kavitha-f5 kavitha-f5 merged commit af71f17 into F5Networks:development May 10, 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