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

Fix the new GitHub Actions tests #2085

Merged
merged 6 commits into from
Feb 13, 2023
Merged

Fix the new GitHub Actions tests #2085

merged 6 commits into from
Feb 13, 2023

Conversation

verdx
Copy link
Contributor

@verdx verdx commented Feb 9, 2023

Related issue
#1626

Description

This PR fixes the GitHub Actions introduced in the mentioned Issue, as one of the functionalities, that of uploading the created docker images when a PR is merged, has been proven to be failing. It can be seen in test https://github.com/wazuh/wazuh-packages/actions/runs/4134303479/jobs/7145279001 for issue #1918.

The changes in this PR have been tested in forked repository https://github.com/verdx/wazuh-packages-testing

The changes made to files debs/build.sh and rpms/build.sh have been introduced so the GitHub Action is run and the docker images for version 4.5 are created and uploaded.

…pull request is merged

Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
Signed-off-by: Miguel Verdaguer Velázquez <verdx@riseup.net>
@verdx verdx self-assigned this Feb 9, 2023
@verdx verdx marked this pull request as ready for review February 10, 2023 08:18
@DFolchA DFolchA merged commit 4f626c6 into 4.5 Feb 13, 2023
@DFolchA DFolchA deleted the fix-upload-docker-images branch February 13, 2023 08:39
@vikman90 vikman90 added type/bug Bug issue and removed type: bug labels Jun 19, 2023
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.

4 participants