Skip to content

[Issue 9622] Fix setting Bookie dbStorage_*CacheMaxSizeMb in pulsar-test-latest-version docker image#9623

Merged
merlimat merged 1 commit intoapache:masterfrom
lhotari:lh-fix-bk-config-in-pulsar-test-docker-image
Feb 19, 2021
Merged

[Issue 9622] Fix setting Bookie dbStorage_*CacheMaxSizeMb in pulsar-test-latest-version docker image#9623
merlimat merged 1 commit intoapache:masterfrom
lhotari:lh-fix-bk-config-in-pulsar-test-docker-image

Conversation

@lhotari
Copy link
Copy Markdown
Member

@lhotari lhotari commented Feb 19, 2021

Fixes #9622

Motivation

When using pulsar-test-latest-version docker image in integration tests, bookies run out of direct memory. This causes integration tests to fail and requiring retries to pass.

There has been intention to set dbStorage_writeCacheMaxSizeMb=16 and dbStorage_readAheadCacheMaxSizeMb=16, however these didn't get applied because of misconfiguration.

See #9622 for long explanation.

Modifications

Fix setting dbStorage_writeCacheMaxSizeMb=16 and dbStorage_readAheadCacheMaxSizeMb=16 when running bookies in pulsar-test-latest-version docker image.

@eolivelli
Copy link
Copy Markdown
Contributor

@codelipenghui @zymap @sijie please merge, this is another step in order to reduce the number of flakes in integration tests

@merlimat merlimat merged commit 1ba7f72 into apache:master Feb 19, 2021
lhotari added a commit that referenced this pull request Feb 16, 2022
…est-latest-version docker image (#9623)

Fixes #9622

(cherry picked from commit 1ba7f72)
@lhotari lhotari added cherry-picked/branch-2.7 Archived: 2.7 is end of life release/2.7.6 labels Feb 16, 2022
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.

Flaky-test: CLITest (and possibly a lot of other integration tests because of the same reason)

5 participants