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.

Sign up for GitHub

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

Jump to bottom

Upgrade Wazuh embedded Python interpreter to 3.10.13 #20003

Merged
merged 71 commits into from Nov 7, 2023
Merged

Upgrade Wazuh embedded Python interpreter to 3.10.13 #20003

merged 71 commits into from Nov 7, 2023

Conversation

fdalmaup
Copy link
Member

@fdalmaup fdalmaup commented Nov 2, 2023 •
edited

Related issue
#13265

Description

Closes #13265. It upgrades the Wazuh embedded Python interpreter from 3.9 to the latest patch version 3.10.13. This includes the upgrade of several Framework and API dependencies such as:

  • aiohttp to 3.8.5
  • connexion to 2.14.2
  • cryptography to 41.0.4
  • SQLAlchemy to 2.0.0
  • uvloop to 0.17.0
  • and others that can be found in the framework/requirements.txt file.

Note: the DEPS_VERSION found in src/Makefile should be modified before merging. Its value corresponds to a development testing version.

Tests

For the complete set of performed tests, please take a look at #13373.

@fdalmaup fdalmaup self-assigned this Nov 2, 2023
@fdalmaup fdalmaup changed the title Wazuh embedded Python interpreter upgrade to 3.10.13 Upgrade Wazuh embedded Python interpreter to 3.10.13 Nov 2, 2023
@fdalmaup fdalmaup linked an issue Nov 2, 2023 that may be closed by this pull request
Upgrade the Wazuh embedded Python to 3.10 #13265
Closed
@Selutario Selutario mentioned this pull request Nov 3, 2023
Closed
@fdalmaup fdalmaup force-pushed the feature/13265-upgrade-python-to-3.10.13 branch 2 times, most recently from 1825c6a to d6cbe50 Compare November 3, 2023 12:10
@GGP1 GGP1 mentioned this pull request Nov 3, 2023
Closed
@fdalmaup fdalmaup marked this pull request as ready for review November 3, 2023 14:29
@fdalmaup fdalmaup mentioned this pull request Nov 3, 2023
Closed
fdalmaup and others added 21 commits November 6, 2023 09:26
@fdalmaup
Update uvloop and cryptography related deps
97e70e6
@fdalmaup
Change WLIBPYTHON to 3.10
75bb6d9
@fdalmaup
Update certifi and future deps
5714402
@fdalmaup
Modify code breaking sections for tests
5f8ace4
@fdalmaup
Update multidict, psutil and yarl deps
c578f15
@fdalmaup
Update cryptography and hiredis dependencies
feb0a83
@fdalmaup
Update aiohttp,certifi,cryptography,grpcio,numpy,pyarrow and requests…
6f1758c 
… deps
@fdalmaup
Update connexion dependency
44170f9
@GGP1 @fdalmaup
Prevent pytest from trying to discover tests in the class
6535f23
@GGP1 @fdalmaup
Use an absolute path
bedcfc1
@GGP1 @fdalmaup
Create custom marker asyncio
687c6a4
@GGP1 @fdalmaup
Replace distutils with setuptools
588f40e
@GGP1 @fdalmaup
Specify the 'loop' parameter
56b94cd
@GGP1 @fdalmaup
Create asyncio event loop
9af8715
@GGP1 @fdalmaup
Remove wrong decorator
43c7ae1
@GGP1 @fdalmaup
Fix collections import
10ee048
@GGP1 @fdalmaup
Rename test functions with returns
58f29c5
@GGP1 @fdalmaup
Update dependencies to merge branch fix/13267-Python3.10-test-warning…
e692913 
…s to feature/13265-upgrade-python-to-3.10.11
@mhamra @fdalmaup
Fix RuntimeWarnings "coroutine 'x' was never awaited" in cluster/core…
3d8048e 
… tests. Issue #13267.
@mhamra @fdalmaup
Fix AbstractClient.connection_result function. Exception Handling. Is…
d572eeb 
…sue #13267
@mhamra @fdalmaup
Fix LocalServerHandler.get_send_file_respones function. Deleted log_e…
555e160 
…xceptions call. Issue #13267
fdalmaup and others added 22 commits November 6, 2023 09:26
@fdalmaup
Update aiohttp,certifi,cryptography,grpcio,numpy,pyarrow and requests…
c36a866 
… deps
@fdalmaup
Modify body type in controller functions of PUT endpoints
0feaaec
@fdalmaup
Modify expected error code
c2c896e
@mhamra @fdalmaup
Remove tracemalloc calls
bbec10d
@mhamra @fdalmaup
Fix connection_made function. Removed call to log_exceptions
9104839
@fdalmaup
Add wheel requirement
6aabc76
@fdalmaup
Bump inflection, multidict, psutil and yarl versions
cb8200c
@fdalmaup
Update inflection
2a8f40e
@fdalmaup
Update SQLAlchemy and related deps
505fa53
@fdalmaup
Modify SQLAlchemy legacy code
6098a8f
@fdalmaup
Adapt legacy SQLAlchemy code
22a7f6f
@fdalmaup
Adapt legacy SQLAlchemy code for version 2.0
d93d2a6
@fdalmaup
Adapt SQLAlchemy legacy code and add pytest fixture teardown
4a0c596
@fdalmaup
Modify DEPS_VERSION for testing
36c538c
@fdalmaup
Remove unnecesary import
0380715
@mhamra @fdalmaup
Change python version from 3.9 to 3.10
8a0bbcb 
in api and framework unit tests workflows.
Add external integrations and wodles unit tests
GitHub workflows
@mhamra @GGP1 @fdalmaup
Update .github/workflows/external-integrations-unit-tests.yml
9d0f27a 
Co-authored-by: GGP1 <51374959+GGP1@users.noreply.github.com>
@mhamra @GGP1 @fdalmaup
Update .github/workflows/external-integrations-unit-tests.yml
b279b10 
Co-authored-by: GGP1 <51374959+GGP1@users.noreply.github.com>
@mhamra @GGP1 @fdalmaup
Update .github/workflows/wodles-unit-tests.yml
d1e985e 
Co-authored-by: GGP1 <51374959+GGP1@users.noreply.github.com>
@mhamra @GGP1 @fdalmaup
Update .github/workflows/wodles-unit-tests.yml
d5b55ca 
Co-authored-by: GGP1 <51374959+GGP1@users.noreply.github.com>
@mhamra @GGP1 @fdalmaup
Update .github/workflows/external-integrations-unit-tests.yml
c80d11a 
Co-authored-by: GGP1 <51374959+GGP1@users.noreply.github.com>
@fdalmaup
Add pytest.ini file for unit tests
7c83990
@fdalmaup fdalmaup force-pushed the feature/13265-upgrade-python-to-3.10.13 branch from d6cbe50 to 7c83990 Compare November 6, 2023 12:26
@fdalmaup fdalmaup force-pushed the feature/13265-upgrade-python-to-3.10.13 branch from 8b89520 to ec8e413 Compare November 7, 2023 11:35
@jotacarma90 jotacarma90 mentioned this pull request Nov 7, 2023
Closed
28 tasks
Selutario
Selutario approved these changes Nov 7, 2023
View reviewed changes
Copy link
Member

@Selutario Selutario left a comment

Choose a reason for hiding this comment

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

LGTM, good job!

@Selutario Selutario merged commit 630e456 into 4.8.0 Nov 7, 2023
106 of 108 checks passed
@Selutario Selutario deleted the feature/13265-upgrade-python-to-3.10.13 branch November 7, 2023 17:35
@fdalmaup fdalmaup mentioned this pull request Nov 14, 2023
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Selutario Selutario Selutario approved these changes

Assignees

@fdalmaup fdalmaup

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Upgrade the Wazuh embedded Python to 3.10
4 participants
@fdalmaup @Selutario @GGP1 @mhamra