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

ci: activate CI for jpa-without-entity integrations native test #5369

Merged

Conversation

machi1990
Copy link
Member

@machi1990 machi1990 commented Nov 11, 2019

This is based on #5343 (comment)

Fixes #5368

/cc @Sanne @geoand @jaikiran

@geoand
Copy link
Contributor

geoand commented Nov 11, 2019

Thanks @machi1990 !

@geoand geoand added this to the 1.1.0 milestone Nov 11, 2019
@geoand geoand added backport? triage/waiting-for-ci Ready to merge when CI successfully finishes labels Nov 11, 2019
@geoand
Copy link
Contributor

geoand commented Nov 11, 2019

Would actually be nice if Azure CI ran this :)

@machi1990
Copy link
Member Author

For some reason CI did not pick it :-(

@geoand
Copy link
Contributor

geoand commented Nov 11, 2019

@machi1990 perhaps you need to also make a dummy "code" change?

@machi1990 machi1990 force-pushed the add-jpa-without-entity-in-native-ci branch from 7f5c45e to a528d70 Compare November 11, 2019 10:24
@machi1990
Copy link
Member Author

@machi1990 perhaps you need to also make a dummy "code" change?

We normally have ci runs for this kind of change. Also, tricking the ci with a code change did not help.
I remember @jmartisk encountered a similar issue where one of his PR was not picked up. Do not remember how was that solved though.

Copy link
Member

@Sanne Sanne left a comment

Choose a reason for hiding this comment

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

great. Thanks!

@machi1990
Copy link
Member Author

Also the merge of #5343 parent PR is encountering ci failures https://dev.azure.com/quarkus-ci/quarkus/_build/results?buildId=11930&view=logs

Information, ApplicationInsightsTelemetrySender will correlate events with X-TFS-Session 2ff6fc4f-b2b8-4380-9c16-2d95323a3454
Information, DedupManifestArtifactClient will correlate http requests with X-TFS-Session 2ff6fc4f-b2b8-4380-9c16-2d95323a3454
Information, Minimatch patterns: [**]
Information, ArtifactHttpRetryMessageHandler.SendAsync: https://vsblobprodsu6weu.vsblob.visualstudio.com/A73d3f287-1d8f-4459-bc46-c7af16e679c5/_apis/dedup/nodes/0C332261DE397A27703EF8006982F80E5637942C0290FFFC5D6753C7DE5250F902 attempt 1/6 failed with StatusCode RedirectMethod, IsRetryableResponse False
Information, Filtered 13816 files from the Minimatch filters supplied.
Information, Could not initialize dataport.
Information, Downloaded 0.0 MB out of 1,127.0 MB (0%).
Information, Downloaded 37.1 MB out of 1,127.0 MB (3%).
Information, Downloaded 256.8 MB out of 1,127.0 MB (23%).
Information, Downloaded 443.5 MB out of 1,127.0 MB (39%).
Information, Downloaded 590.1 MB out of 1,127.0 MB (52%).
Information, Downloaded 695.4 MB out of 1,127.0 MB (62%).
Information, Downloaded 800.5 MB out of 1,127.0 MB (71%).
Information, Downloaded 872.4 MB out of 1,127.0 MB (77%).
Information, ArtifactHttpRetryMessageHandler.SendAsync: https://vsblobprodsu6weu.vsblob.visualstudio.com/A73d3f287-1d8f-4459-bc46-c7af16e679c5/_apis/dedup/urls attempt 1/6 failed with HttpRequestException: 'An error occurred while sending the request.', HttpStatusCode 
Information, ArtifactHttpRetryMessageHandler.SendAsync: https://vsblobprodsu6weu.vsblob.visualstudio.com/A73d3f287-1d8f-4459-bc46-c7af16e679c5/_apis/dedup/urls attempt 1/6 throwing HttpRequestException with AttemptCount 1, LastBackoff 10000ms
Information, Downloaded 1,104.2 MB out of 1,127.0 MB (98%).
Information, ArtifactHttpRetryMessageHandler.SendAsync: https://vsblobprodsu6weu.vsblob.visualstudio.com/A73d3f287-1d8f-4459-bc46-c7af16e679c5/_apis/dedup/urls attempt 1/6 failed with HttpRequestException: 'An error occurred while sending the request.', HttpStatusCode 
Information, ArtifactHttpRetryMessageHandler.SendAsync: https://vsblobprodsu6weu.vsblob.visualstudio.com/A73d3f287-1d8f-4459-bc46-c7af16e679c5/_apis/dedup/urls attempt 1/6 throwing HttpRequestException with AttemptCount 1, LastBackoff 10000ms
Information, Downloaded 1,105.9 MB out of 1,127.0 MB (98%).
Information, Downloaded 1,123.0 MB out of 1,127.0 MB (100%).
Information, ApplicationInsightsTelemetrySender correlated 2 events with X-TFS-Session 2ff6fc4f-b2b8-4380-9c16-2d95323a3454
##[error]One or more errors occurred. (One or more errors occurred. (An error occurred while sending the request.))
##[section]Finishing: DownloadPipelineArtifact

something to do with Azure maybe?

@machi1990
Copy link
Member Author

Finally picked up by ci and all green. Let's merge this. Thanks @Sanne @geoand

@machi1990 machi1990 merged commit 3200d9c into quarkusio:master Nov 11, 2019
@machi1990 machi1990 deleted the add-jpa-without-entity-in-native-ci branch November 11, 2019 16:10
@gsmet gsmet removed the backport? label Nov 14, 2019
@gsmet gsmet modified the milestones: 1.1.0, 1.0.0.Final Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Run native-image tests for quarkus-integration-test-jpa-without-entity on CI
4 participants