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

Add a new method for obtaining S3 packages #1828

Merged
merged 10 commits into from
Sep 3, 2021

Conversation

fernandolojano
Copy link
Contributor

Related issue
#1809

Description

It is necessary to implement new functionalities in order to get a new alternative method for getting the packages from the S3 server.

This PR makes the following changes:

  • Add new fields and conditional requirements in the schema validator for accepting the needed parameters of the new functionality.

  • Add a new tool for generating s3 package URLs called s3_package.py

  • Modify the QAProvisioning implementation in order to add a new alternative method for getting the s3 package if needed.

  • There is a new tool now called s3_package.py

  • There is a new way to get the s3 packages if needed

  • The schema validator now validates new fields

Tasks

The tasks made for achieving this new implementation are listed in the issue itself.
However, the last changes are commented in this comment.

Copy link
Contributor

@jmv74211 jmv74211 left a comment

Choose a reason for hiding this comment

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

Good job.

Quite a few changes are required. I'll contact you for a more in depth review.

deps/wazuh_testing/wazuh_testing/tools/s3_package.py Outdated Show resolved Hide resolved
deps/wazuh_testing/wazuh_testing/tools/s3_package.py Outdated Show resolved Hide resolved
deps/wazuh_testing/wazuh_testing/tools/s3_package.py Outdated Show resolved Hide resolved
deps/wazuh_testing/wazuh_testing/tools/s3_package.py Outdated Show resolved Hide resolved
deps/wazuh_testing/wazuh_testing/tools/s3_package.py Outdated Show resolved Hide resolved
@jmv74211 jmv74211 force-pushed the 1809-gen-s3-links-implementation branch from bcadf85 to 2723fb6 Compare September 2, 2021 15:48
@jmv74211 jmv74211 merged commit 01b44da into 1533-qa-ctl Sep 3, 2021
@jmv74211 jmv74211 deleted the 1809-gen-s3-links-implementation branch September 3, 2021 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants