Skip to content

Consumer missed part fix#10771

Merged
oscerd merged 3 commits into
apache:mainfrom
hamed-hatami:consumer-missed-part-fix
Jul 21, 2023
Merged

Consumer missed part fix#10771
oscerd merged 3 commits into
apache:mainfrom
hamed-hatami:consumer-missed-part-fix

Conversation

@hamed-hatami
Copy link
Copy Markdown
Contributor

@hamed-hatami hamed-hatami commented Jul 21, 2023

Description

Target

  • I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas 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 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! 🌟

🐫 Maintainers, please note that first-time contributors require manual approval for the GitHub Actions to run.

⚠️ Please note that the changes on this PR may be tested automatically if they change components.

🤖 Use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot.

If necessary Apache Camel Committers may access logs and test results in the job summaries!

@hamed-hatami
Copy link
Copy Markdown
Contributor Author

Is this the same issue , we saw for doc update too ?

@hamed-hatami
Copy link
Copy Markdown
Contributor Author

Locally all UT and IT got green

@oscerd
Copy link
Copy Markdown
Contributor

oscerd commented Jul 21, 2023

Same. @orpiske @davsclaus I think we should have a look at incremental test

@hamed-hatami
Copy link
Copy Markdown
Contributor Author

Is there any workaround to merge for the time being ?

@oscerd
Copy link
Copy Markdown
Contributor

oscerd commented Jul 21, 2023

I tested the PR locally, so I'm going to merge.

@oscerd oscerd merged commit 78b29b5 into apache:main Jul 21, 2023
@orpiske
Copy link
Copy Markdown
Contributor

orpiske commented Jul 21, 2023

Same. @orpiske @davsclaus I think we should have a look at incremental test

There is something outdated on GH:

ERROR] Failed to execute goal on project camel-allcomponents: Could not resolve dependencies for project org.apache.camel:camel-allcomponents:pom:4.0.0-SNAPSHOT: Failed to collect dependencies at org.apache.camel:camel-jira:jar:4.0.0-SNAPSHOT -> io.atlassian.fugue:fugue:jar:5.0.0: Failed to read artifact descriptor for io.atlassian.fugue:fugue:jar:5.0.0: The following artifacts could not be resolved: io.atlassian.fugue:fugue-parent:pom:5.0.0 (present, but unavailable): io.atlassian.fugue:fugue-parent:pom:5.0.0 was not found in https://repo1.maven.org/maven2/ during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal on project camel-allcomponents: Could not resolve dependencies for project org.apache.camel:camel-allcomponents:pom:4.0.0-SNAPSHOT: Failed to collect dependencies at org.apache.camel:camel-jira:jar:4.0.0-SNAPSHOT -> io.atlassian.fugue:fugue:jar:5.0.0

@davsclaus
Copy link
Copy Markdown
Contributor

davsclaus commented Jul 21, 2023

camel-allcomponents has been added the atlassian repo 30 min ago or so

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.

4 participants