Skip to content

feat(dsl): camel jbang export jib#24072

Open
squakez wants to merge 1 commit into
apache:mainfrom
squakez:feat/CAMEL-23780
Open

feat(dsl): camel jbang export jib#24072
squakez wants to merge 1 commit into
apache:mainfrom
squakez:feat/CAMEL-23780

Conversation

@squakez

@squakez squakez commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Added a Jib profile to enable the publishing of the container via Jib profile

NOTE: this is separate from the existing jib flags which may be used in kubernetes plugin (via jkube and others).

Closes CAMEL-23780

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.

@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 github-actions Bot added the dsl label Jun 17, 2026

@davsclaus davsclaus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use the same jib replacholder as in the k8s

              <version>[=JibMavenPluginVersion]</version>

@squakez squakez force-pushed the feat/CAMEL-23780 branch from ba5fce3 to d9cfb9a Compare June 17, 2026 09:39
@squakez

squakez commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

JibMavenPluginVersion

I don't think so. That variable is enabled when the jkube plugin is enabled. The goal of this PR is separate from the Jkube one as it wants to include the plugin regardless of the presence of other tools. The idea is that the user can launch a mvn jib:build OOTB and have the same experience with all the runtimes we support as explained also in the readme.md files of each export.

@davsclaus

Copy link
Copy Markdown
Contributor

Yeah but we should not hardcode the version, so it does not go stale

@squakez

squakez commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

Yeah but we should not hardcode the version, so it does not go stale

Ah, okey. Sure, I'll see how to make it a variable then. Thanks!

@github-actions

Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • dsl/camel-jbang/camel-jbang-core

⚠️ Some tests are disabled on GitHub Actions (@DisabledIfSystemProperty(named = "ci.env.name")) and require manual verification:

  • dsl/camel-jbang/camel-jbang-core: 1 test(s) disabled on GitHub Actions

💡 Manual integration tests recommended:

You modified dsl/camel-jbang/camel-jbang-core. The related integration tests in dsl/camel-jbang/camel-jbang-it are excluded from CI. Consider running them manually:

mvn verify -f dsl/camel-jbang/camel-jbang-it -Djbang-it-test
All tested modules (6 modules)
  • Camel :: JBang :: Core
  • Camel :: JBang :: MCP
  • Camel :: JBang :: Plugin :: Route Parser
  • Camel :: JBang :: Plugin :: TUI
  • Camel :: JBang :: Plugin :: Validate
  • Camel :: Launcher :: Container

⚙️ View full build and test results

@squakez squakez force-pushed the feat/CAMEL-23780 branch from d9cfb9a to 6307e91 Compare June 17, 2026 12:25
@squakez squakez force-pushed the feat/CAMEL-23780 branch from 6307e91 to a3a5654 Compare June 17, 2026 13:10
@github-actions github-actions Bot added the docs label Jun 17, 2026
@squakez squakez force-pushed the feat/CAMEL-23780 branch 2 times, most recently from 2dc6e93 to 4208906 Compare June 17, 2026 13:48
Added a Jib profile to enable the publishing of the container via Jib profile

Closes CAMEL-23780
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