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

Merge 4.8.0 into 4.8.1 #2895

Merged
merged 10 commits into from
Apr 1, 2024
Merged

Merge 4.8.0 into 4.8.1 #2895

merged 10 commits into from
Apr 1, 2024

Conversation

vikman90
Copy link
Member

@vikman90 vikman90 commented Apr 1, 2024

Related issue
#2894

asteriscos and others added 10 commits March 7, 2024 15:44
@asteriscos
Change dashboard-management menu order
8be27eb
@asteriscos
Merge pull request #2873 from wazuh/change/dashboard-management-categ…
b943ef9 
…ory-order-6483

Change dashboard management category order
@sebasfalcone
CL:
c93e469 
- Updated permissions for:
  - /var/ossec/templates
  - /var/ossec/queue/vd
  - /var/ossec/queue/indexer
@MarcelKemp
Skip certificate validation for CentOS 5
a2e3825
@MarcelKemp
Merge pull request #2891 from wazuh/2889-fix-ssl-verification
efc5d8a 
Skip certificate validation for CentOS 5 package generation
@teddytpc1
Port variable name fixed
c80b685
@teddytpc1
Merge pull request #2892 from wazuh/2888-error-port-443-is-being-used…
0f3a455 
…-by-another-process

Port variable name fixed
@teddytpc1
Merge pull request #2893 from wazuh/merge-4.7.3-into-4.8.0
7c58227 
Merge 4.7.3 into 4.8.0
@Dwordcito
Merge pull request #2890 from wazuh/dev-6363-update-permissions-on-ma…
39752e3 
…nager-specs

Update VD file permissions
@vikman90
Merge branch '4.8.0' into merge-4.8.0-into-4.8.1
24d5933
@vikman90 vikman90 changed the base branch from master to 4.8.1 April 1, 2024 07:56
@vikman90 vikman90 self-assigned this Apr 1, 2024
@vikman90 vikman90 requested a review from MarcelKemp April 1, 2024 07:56
@vikman90 vikman90 linked an issue Apr 1, 2024 that may be closed by this pull request
Scheduled upward merges for numbered branches in wazuh-packages repository - Fortnight #7 #2894
Closed
5 tasks
@vikman90 vikman90 mentioned this pull request Apr 1, 2024
Closed
5 tasks
@MarcelKemp
Copy link
Member

Failed checks

Distributes cases and All in One tests

The PR checks are failing due to an error detection in the manager logs after installation:

wazuh-modulesd:vulnerability-scanner: ERROR: Error processing message: [json.exception.out_of_range.401] array index 8 is out of range

This bug comes from 4.8.0 and is already reported in the following issue, which seems to have a PR where the problem is fixed:

Build Wazuh packages

The checks fail because the base branch was initially set to master by mistake, causing it to attempt to download and install a 5.0.0 package.

This failure is expected because of the way the test workflow is set up, and it would simply take a new commit to take the 4.8.1 branch and build it correctly.

@MarcelKemp MarcelKemp merged commit f5c979e into 4.8.1 Apr 1, 2024
48 of 68 checks passed
@MarcelKemp MarcelKemp deleted the merge-4.8.0-into-4.8.1 branch April 1, 2024 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MarcelKemp MarcelKemp MarcelKemp approved these changes

Assignees

@vikman90 vikman90

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Scheduled upward merges for numbered branches in wazuh-packages repository - Fortnight #7
6 participants
@vikman90 @MarcelKemp @asteriscos @sebasfalcone @teddytpc1 @Dwordcito