Skip to content

CAMEL-23499 - Disable again several FileToFtps* tests#23327

Merged
apupier merged 1 commit into
apache:mainfrom
apupier:23499-DisableFtpTestWhichAreNeverPassingOnCI
May 19, 2026
Merged

CAMEL-23499 - Disable again several FileToFtps* tests#23327
apupier merged 1 commit into
apache:mainfrom
apupier:23499-DisableFtpTestWhichAreNeverPassingOnCI

Conversation

@apupier
Copy link
Copy Markdown
Contributor

@apupier apupier commented May 19, 2026

Several FTP tests were re-enabled recently with
#23078 but unfortunately several are broken. They always fail on CI or when launching from Maven locally (at least for me)
Also note that some of them are disabled in case the JVM does not support specific algorithm, which means that they are not raising an error in this case.

These tests are passing if I launch them from my IDE.

Description

Target

  • I checked that the commit is targeting the correct branch (Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

Several FTP tests were re-enabled recently with
apache#23078 but unfortunately several are
broken. They always fail on CI or when launching from Maven locally (at
least for me)
Also note that some of them are disabled in case the JVM does not
support specific algorithm, which means that they are not raising an
error in this case.

Signed-off-by: Aurélien Pupier <apupier@ibm.com>
@apupier apupier marked this pull request as ready for review May 19, 2026 15:34
@apupier apupier merged commit 4166cfa into apache:main May 19, 2026
6 checks passed
@apupier apupier deleted the 23499-DisableFtpTestWhichAreNeverPassingOnCI branch May 19, 2026 15:34
@apupier
Copy link
Copy Markdown
Contributor Author

apupier commented May 19, 2026

note that it will be nice to unskip all the skipped tests one day but better to do it without impacting the main branch

@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions
Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • components/camel-ftp
All tested modules (11 modules)
  • Camel :: Azure :: Files
  • Camel :: FTP
  • Camel :: JBang :: MCP
  • Camel :: JBang :: Plugin :: Route Parser
  • Camel :: JBang :: Plugin :: TUI
  • Camel :: JBang :: Plugin :: Validate
  • Camel :: Jsch
  • Camel :: Launcher :: Container
  • Camel :: YAML DSL :: Validator
  • Camel :: YAML DSL :: Validator Maven Plugin
  • Camel :: Zookeeper Master

⚙️ View full build and test results

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.

2 participants