Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AWS 2 Lambda native support #1572

Closed
ppalaga opened this issue Aug 24, 2020 · 10 comments
Closed

AWS 2 Lambda native support #1572

ppalaga opened this issue Aug 24, 2020 · 10 comments

Comments

@ppalaga
Copy link
Contributor

ppalaga commented Aug 24, 2020

No description provided.

@Jeansen
Copy link
Contributor

Jeansen commented Aug 24, 2020

I think this one is quite simple to do, I'd give it a try this or next weekend.

@ppalaga
Copy link
Contributor Author

ppalaga commented Aug 25, 2020

That's highly welcome, unless @oscerd has something in the works?

@oscerd
Copy link
Contributor

oscerd commented Aug 25, 2020

Hello @Jeansen

Feel free to work on this extension :-)

Thanks a lot for your time!

@Jeansen
Copy link
Contributor

Jeansen commented Sep 3, 2020

I am sorry, this the passed WE I did not find time. I hope this WE works out. This time, it will - sadly - not be done in 9 days ;-)

@Jeansen
Copy link
Contributor

Jeansen commented Sep 7, 2020

When promoting to native, I got some error regarding the POM.xml file in the integration-tests folder

[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]   
[ERROR]   The project  (/home/marcel/ownCloud/workspaces/projects/camel-quarkus/integration-tests/pom.xml) has 1 error
[ERROR]     Non-parseable POM /home/marcel/ownCloud/workspaces/projects/camel-quarkus/integration-tests/pom.xml: Duplicated tag: 'modules' (position: START_TAG seen ...</dependencyManagement>\n\n    <modules>... @53:14)  @ line 53, column 14 -> [Help 2]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/ModelParseException

And in the POM, I see

 <modules>
        <module>aws2-lambda</module>
    </modules>

    <properties>
        <quarkus.banner.enabled>false</quarkus.banner.enabled>
    </properties>

    <dependencyManagement>
        <dependencies>
            <dependency>
                <groupId>io.quarkus</groupId>
                <artifactId>quarkus-junit5-internal</artifactId>
                <version>${quarkus.version}</version>
            </dependency>
        </dependencies>
    </dependencyManagement>

    <modules>
        <!-- build those first -->
        <module>core</module>
        <module>main</module>
        <module>main-devmode</module>
        <module>main-xml-jaxb</module>
        <module>main-xml-io</module>
        <module>main-collector</module>
        <module>main-caffeine-lrucache</module>
        <module>main-command-mode</module>
.
.
.

Obviously, something with the module addition went wrong ;-) I'll fix it manually for now.

@Jeansen
Copy link
Contributor

Jeansen commented Sep 7, 2020

As I see, integration tests are all in one folder. So, I assume I do not have to amend the tooling/scripts/test-categories.yaml file?
Is there any automation? Or should I just manually throw away what I do not need from the generated integration test and simply try adding a rout in the aws2 test class?

@oscerd
Copy link
Contributor

oscerd commented Sep 7, 2020

You should add just the needed bits into the camel-aws2 test folder. It should be enough to add the extension and write a similar route.

@Jeansen
Copy link
Contributor

Jeansen commented Sep 7, 2020

OK, thx.

@ppalaga
Copy link
Contributor Author

ppalaga commented Sep 7, 2020

Obviously, something with the module addition went wrong

Thanks for complaining :), let me check that tomorrow.

@ppalaga
Copy link
Contributor Author

ppalaga commented Sep 8, 2020

Obviously, something with the module addition went wrong

Thanks for complaining :), let me check that tomorrow.

Should be fixed via f4233ff

Jeansen added a commit to Jeansen/camel-quarkus that referenced this issue Sep 8, 2020
Jeansen added a commit to Jeansen/camel-quarkus that referenced this issue Sep 8, 2020
Jeansen added a commit to Jeansen/camel-quarkus that referenced this issue Sep 8, 2020
Jeansen added a commit to Jeansen/camel-quarkus that referenced this issue Sep 8, 2020
@ppalaga ppalaga closed this as completed in e8fb61a Sep 8, 2020
@jamesnetherton jamesnetherton added this to the 1.1.0 milestone Sep 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants