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

bump es heap sizes for integration tests #14543

Merged
merged 3 commits into from
Nov 15, 2019

Conversation

graphaelli
Copy link
Member

@graphaelli graphaelli commented Nov 15, 2019

matching the defaults. 512 MB was set in #2947 back in 2016 without discussion so I'm unaware of any special concerns around this.

EDIT: now also downloads jolokia-war.war from maven central and verifies its contents

@graphaelli graphaelli requested a review from a team as a code owner November 15, 2019 19:00
also separate tomcat and retry steps for some layer caching
Copy link
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

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

Thanks!

@jsoriano jsoriano added :Testing needs_backport PR is waiting to be backported to other branches. review labels Nov 15, 2019
@graphaelli
Copy link
Member Author

It seems the only failure was due to:

21:21:06 compose.service: ERROR: 404 Client Error: Not Found ("pull access denied for docker.elastic.co/observability-ci/beats-integration-couchbase, repository does not exist or may require 'docker login': denied: requested access to the resource is denied")```

@graphaelli graphaelli merged commit 08d9acf into elastic:master Nov 15, 2019
@graphaelli graphaelli deleted the more-es-mem branch November 15, 2019 22:23
graphaelli added a commit to graphaelli/beats that referenced this pull request Nov 15, 2019
…ainer image (elastic#14543)

* bump es heap sizes for integration tests

* update path to jolokia-war.war

* get jolokia-war from maven and verify content

also separate tomcat and retry steps for some layer caching
@jsoriano jsoriano removed the needs_backport PR is waiting to be backported to other branches. label Nov 15, 2019
graphaelli added a commit to graphaelli/beats that referenced this pull request Nov 15, 2019
    * get jolokia-war from maven and verify content

also separate tomcat and retry steps for some layer caching
graphaelli added a commit to graphaelli/beats that referenced this pull request Nov 15, 2019
    * get jolokia-war from maven and verify content

also separate tomcat and retry steps for some layer caching
graphaelli added a commit that referenced this pull request Nov 16, 2019
* get jolokia-war from maven and verify content

also separate tomcat and retry steps for some layer caching
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
* get jolokia-war from maven and verify content

also separate tomcat and retry steps for some layer caching
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