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

[Ops] POC: PR-build on CI-Systems buildkite (cont'd) #171317

Closed
wants to merge 82 commits into from

Conversation

delanni
Copy link
Contributor

@delanni delanni commented Nov 15, 2023

Summary

This doesn't need to be merged, it's a working POC.

Continued, refreshed work on trying to run crucial build jobs on the CI-systems pipelines.
Continuation of #157220

KME test runs here: https://buildkite.com/elastic/kibana-kme-test

Main issue: https://github.com/elastic/kibana-operations/issues/22
Meta issue: https://github.com/elastic/kibana-operations/issues/15

Remaining problems:

brianseeders and others added 2 commits November 15, 2023 16:39
WIP prep things to test elastic buildkite system and without secrets

Make vars exist but be empty

Switch to n1s for now

Output pipeline instead of upload

Comment out function

A few more fixes

Bring back the secrets

Fix paths

Fix path

Bring back all the FTR configs

Use util

Fix dynamic agents

n1 -> n2

Fix commented out yaml

Add umask debug step

Test commit

test commit

test commit

test commit

Move to new image family

Remove umask step

Remove commented out section

[CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

Test new git config

Private repo instead

Make agents spot instances

Update a few missed agent configs
@delanni
Copy link
Contributor Author

delanni commented Nov 16, 2023

buildkite build this

@delanni delanni force-pushed the buildkite-on-elastic-ci-poc branch from 6c2405c to c4de3bd Compare November 16, 2023 22:12
@delanni
Copy link
Contributor Author

delanni commented Nov 16, 2023

buildkite build this

@delanni delanni force-pushed the buildkite-on-elastic-ci-poc branch from 57623d1 to efd5aa3 Compare December 5, 2023 20:41
@delanni delanni force-pushed the buildkite-on-elastic-ci-poc branch from 9f068b9 to eda32dc Compare February 7, 2024 11:28
@delanni delanni force-pushed the buildkite-on-elastic-ci-poc branch from eda32dc to b98ecd8 Compare February 7, 2024 11:29
@delanni delanni force-pushed the buildkite-on-elastic-ci-poc branch from 7e78e04 to 24bd3b1 Compare February 9, 2024 17:14
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Feb 15, 2024
…ic#174688)

## Summary
Creates a CLI script to rewrite buildkite agent targeting rules to the
ones we need to use with the Elastic-wide buildkite agents.

This version tries to map all fittable fields from
https://github.com/elastic/kibana-buildkite/blob/main/agents.json to
https://github.com/elastic/ci/blob/0df8430357109a19957dcfb1d867db9cfdd27937/docs/gobld/providers.mdx#L96

Based on: elastic#171317 (demonstrated here:
elastic@ca0f455)
Part of: elastic/kibana-operations#15
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Feb 15, 2024
## Summary
On the new Buildkite infra, our agents won't have write access to the
paths we used to write the deployment information upon deployment.

We're allowed to use KV writes if we enable it
(https://docs.elastic.dev/ci/using-secrets#using-shared-secrets,
enabling PRs below).

I've built this in a way that we can enable the feature before the final
rollout, and we can clear up the branches once done with the rollout.

Cloud deployment works on the old infra as well as the new, tested on
this PR and elastic#171317

Enabled by: elastic/ci#2594 &
elastic/ci#2553
Part of: elastic/kibana-operations#15
Related: https://elasticco.atlassian.net/browse/ENGPRD-414
@delanni delanni force-pushed the buildkite-on-elastic-ci-poc branch from af3cc1a to 24bd3b1 Compare February 20, 2024 13:54
@delanni
Copy link
Contributor Author

delanni commented May 10, 2024

buildkite build this

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

History

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

@delanni
Copy link
Contributor Author

delanni commented Jun 25, 2024

This was just a POC, it's done in: #183158

@delanni delanni closed this Jun 25, 2024
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.

3 participants