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

[FIPS / CI] Fix ES ML startup issues, UUID permissions, FTR tests status, OpenSSL target. Switch to Ubuntu #182295

Merged
merged 29 commits into from
May 16, 2024

Conversation

Ikuni17
Copy link
Contributor

@Ikuni17 Ikuni17 commented May 1, 2024

Closes elastic/kibana-operations#96

new pipeline run
old pipeline run

  • Fixes OpenSSL compilation so that it does not overwrite the OS version and break OS functionality.
  • Fixes issue where ES would not start due to ML pipe being unable to write to Vagrant synced folder.
    • Reenabled ML in FIPS pipeline
  • Fixes permission where Kibana could not write UUID file.
  • Fixes smoke test exit code not reporting correctly.
  • Fixes Buildkite annotation for failed tests example.
  • Switches the base VM image from RHEL9 to Ubuntu due to RHEL9 subscription requirements to install repo packages.
    • This blocked installing Chrome, causing tests using Webdriver to fail.

This reverts commit 803945c.
@Ikuni17 Ikuni17 self-assigned this May 1, 2024
@Ikuni17
Copy link
Contributor Author

Ikuni17 commented May 2, 2024

/ci

@Ikuni17
Copy link
Contributor Author

Ikuni17 commented May 10, 2024

/ci

@Ikuni17 Ikuni17 changed the title [FIPS] Fix ES ML startup issues, UUID permissions, FTR tests status [FIPS / CI] Fix ES ML startup issues, UUID permissions, FTR tests status, OpenSSL target. Switch to Ubuntu May 10, 2024
@Ikuni17 Ikuni17 marked this pull request as ready for review May 10, 2024 20:18
@Ikuni17 Ikuni17 requested a review from a team as a code owner May 10, 2024 20:18
@Ikuni17 Ikuni17 requested a review from a team as a code owner May 10, 2024 20:18
@Ikuni17 Ikuni17 added Team:Operations Team label for Operations Team release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting labels May 10, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@Ikuni17
Copy link
Contributor Author

Ikuni17 commented May 10, 2024

/ci

Copy link
Member

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

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

LGTM, left a comment about description and possible code change to read variable only once

Copy link
Member

@jbudz jbudz left a comment

Choose a reason for hiding this comment

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

I see there's an issue for tracking test parallelization so wanted to +1 that. The test run is 6 hours, which is going to make for a slow feedback loop on release builds.

@Ikuni17 Ikuni17 requested a review from dmlemeshko May 14, 2024 01:13
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #61 / serverless search UI Importing an existing dashboard should render all panels on the dashboard

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @Ikuni17

Copy link
Member

@dmlemeshko dmlemeshko left a comment

Choose a reason for hiding this comment

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

LGTM

@Ikuni17 Ikuni17 merged commit e53ff44 into elastic:main May 16, 2024
19 checks passed
@Ikuni17 Ikuni17 deleted the fix/vagrant-fips-kibana-uuid branch May 16, 2024 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants