Skip to content

Fix product tests previously migrated to native fs to not use legacy fs#22423

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
anusudarsan:anu/migrate-pending-nfs-tests-to-not-use-hdfs
Jun 20, 2024
Merged

Fix product tests previously migrated to native fs to not use legacy fs#22423
ebyhr merged 1 commit intotrinodb:masterfrom
anusudarsan:anu/migrate-pending-nfs-tests-to-not-use-hdfs

Conversation

@anusudarsan
Copy link
Copy Markdown
Member

@anusudarsan anusudarsan commented Jun 18, 2024

Description

Tests that were migrated to native fs was still using hdfs. So migrating them completely to use native-fs

Additional context and related issues

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Jun 18, 2024
@anusudarsan
Copy link
Copy Markdown
Member Author

@ebyhr can you trigger ci with secrets please?

@anusudarsan anusudarsan force-pushed the anu/migrate-pending-nfs-tests-to-not-use-hdfs branch from a3fa743 to 6289214 Compare June 18, 2024 16:45
@ebyhr
Copy link
Copy Markdown
Member

ebyhr commented Jun 18, 2024

/test-with-secrets sha=62892148a69e36791a920375a7686a539f5e601a

@github-actions
Copy link
Copy Markdown

The CI workflow run with tests that require additional secrets has been started: https://github.com/trinodb/trino/actions/runs/9573025983

@anusudarsan anusudarsan force-pushed the anu/migrate-pending-nfs-tests-to-not-use-hdfs branch from 6289214 to 5a87933 Compare June 20, 2024 17:44
@anusudarsan
Copy link
Copy Markdown
Member Author

@ebyhr addressed comments

@ebyhr ebyhr merged commit 3e8f071 into trinodb:master Jun 20, 2024
@github-actions github-actions bot added this to the 451 milestone Jun 20, 2024
@anusudarsan anusudarsan deleted the anu/migrate-pending-nfs-tests-to-not-use-hdfs branch June 21, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants