Skip to content

Delete unused antalya directory - #2348

Merged
arthurpassos merged 1 commit into
antalya-26.6from
arthurpassos-patch-15
Sep 11, 2026
Merged

Delete unused antalya directory#2348
arthurpassos merged 1 commit into
antalya-26.6from
arthurpassos-patch-15

Conversation

@arthurpassos

@arthurpassos arthurpassos commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Changelog category (leave one):

  • Documentation (changelog entry is not required)

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

Remove the outdated set of documentation. The up-to-date Antalya documentation lives exclusively in docs/en/antalya from now on.

Documentation entry for user-facing changes

...

CI/CD Options

Exclude tests:

  • Fast test
  • Integration Tests
  • Stateless tests
  • Stateful tests
  • Unit tests
  • Performance tests
  • Aarch64 tests
  • All with ASAN
  • All with TSAN
  • All with MSAN
  • All with UBSAN
  • All with Coverage
  • All Regression
  • Disable CI Cache

Regression jobs to run:

  • Fast suites (mostly <1h)
  • Aggregate Functions (2h)
  • Alter (1.5h)
  • Benchmark (30m)
  • CAS (content-addressed storage; Antalya only)
  • ClickHouse Keeper (1h)
  • Iceberg (2h)
  • LDAP (1h)
  • OAuth (5m)
  • Parquet (1.5h)
  • RBAC (1.5h)
  • SSL Server (1h)
  • S3 (2h)
  • S3 Export (2h)
  • Swarms (30m)
  • Tiered Storage (2h)

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-11T01:47:36.022066Z 9344937 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions

Copy link
Copy Markdown

Workflow [PR], commit [9344937]

@mkmkme

mkmkme commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Could you reword the changelog entry please? Now it looks like a justification for a removal. And in the release notes, it's going to look like this:

These set of docs are not up to date, not in the right place and there is a substitute under docs/en/antalya. LLM tools read this and make wrong assumptions (#2348 by @arthurpassos)

Instead it should describe the change itself. I would pick something like this:

Remove the outdated set of documentation. The up-to-date Antalya documentation lives exclusively in docs/en/antalya from now on.

@arthurpassos

Copy link
Copy Markdown
Collaborator Author

Could you reword the changelog entry please? Now it looks like a justification for a removal. And in the release notes, it's going to look like this:

These set of docs are not up to date, not in the right place and there is a substitute under docs/en/antalya. LLM tools read this and make wrong assumptions (#2348 by @arthurpassos)

Instead it should describe the change itself. I would pick something like this:

Remove the outdated set of documentation. The up-to-date Antalya documentation lives exclusively in docs/en/antalya from now on.

Yes, that's better. Thanks

@arthurpassos arthurpassos added antalya port-antalya PRs to be ported to all new Antalya releases antalya-26.6 labels Sep 11, 2026
@arthurpassos
arthurpassos merged commit 0ccff1e into antalya-26.6 Sep 11, 2026
248 of 266 checks passed
@mkmkme mkmkme added the verified Approved for release label Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

antalya antalya-26.6 port-antalya PRs to be ported to all new Antalya releases verified Approved for release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants