Skip to content

Install tailscale during AMI build and set it up on runners#60316

Merged
Felixoid merged 3 commits intomasterfrom
tailscale-ami
Feb 29, 2024
Merged

Install tailscale during AMI build and set it up on runners#60316
Felixoid merged 3 commits intomasterfrom
tailscale-ami

Conversation

@Felixoid
Copy link
Copy Markdown
Member

@Felixoid Felixoid commented Feb 22, 2024

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Preparation for tailscale setup

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Feb 22, 2024
@robot-clickhouse
Copy link
Copy Markdown
Member

robot-clickhouse commented Feb 22, 2024 •
edited
Loading

This is an automated comment for commit ff35382 with description of existing statuses. It's updated for the latest CI running

✅ Click here to open a full report in a separate page

Successful checks
Check nameDescriptionStatus
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR✅ success
ClickBenchRuns [ClickBench](https://github.com/ClickHouse/ClickBench/) with instant-attach table✅ success
ClickHouse build checkBuilds ClickHouse in various configurations for use in further steps. You have to fix the builds that fail. Build logs often has enough information to fix the error, but you might have to reproduce the failure locally. The cmake options can be found in the build log, grepping for cmake. Use these options and follow the general build process✅ success
Compatibility checkChecks that clickhouse binary runs on distributions with old libc versions. If it fails, ask a maintainer for help✅ success
Docker keeper imageThe check to build and optionally push the mentioned image to docker hub✅ success
Docker server imageThe check to build and optionally push the mentioned image to docker hub✅ success
Docs checkBuilds and tests the documentation✅ success
Install packagesChecks that the built packages are installable in a clear environment✅ success
Mergeable CheckChecks if all other necessary checks are successful✅ success
SQLTestThere's no description for the check yet, please add it to tests/ci/ci_config.py:CHECK_DESCRIPTIONS✅ success
Stateful testsRuns stateful functional tests for ClickHouse binaries built in various configurations -- release, debug, with sanitizers, etc✅ success
Style checkRuns a set of checks to keep the code style clean. If some of tests failed, see the related log from the report✅ success
Unit testsRuns the unit tests for different release types✅ success

@Felixoid Felixoid added the do not test disable testing on pull request label Feb 22, 2024
@Felixoid
Copy link
Copy Markdown
Member Author

Felixoid commented Feb 23, 2024

TODO:

  • Chmod the script. Probably, move it separately and install via TOE
  • Move the INSTANCE_ID inside. Setup the suffix to identify a runner type.

@Felixoid Felixoid mentioned this pull request Feb 25, 2024
Merged
@Felixoid Felixoid marked this pull request as ready for review February 29, 2024 16:47
@Felixoid Felixoid merged commit af63125 into master Feb 29, 2024
@Felixoid Felixoid deleted the tailscale-ami branch February 29, 2024 16:47
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

do not test disable testing on pull request pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Felixoid @robot-clickhouse @robot-clickhouse-ci-2