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

Feature/92628-rmq-shovel-plugin #403

Merged
merged 3 commits into from
Jul 11, 2024
Merged

Conversation

hlebkanonik
Copy link
Contributor

@hlebkanonik hlebkanonik commented Jul 11, 2024

Removed:

  • RP_AMQP_QUEUES
  • RP_AMQP_QUEUESPERPOD

Added:

  • REPORTING_QUEUES_COUNT

@hlebkanonik hlebkanonik self-assigned this Jul 11, 2024
@hlebkanonik hlebkanonik merged commit 8ff681f into develop Jul 11, 2024
@hlebkanonik hlebkanonik deleted the feature/92628-rmq-shovel-plugin branch July 11, 2024 15:40
@@ -155,13 +155,8 @@ serviceapi:
jvmArgs: "-Djava.security.egd=file:/dev/./urandom -XX:+UseG1GC -XX:MinRAMPercentage=60.0 -XX:InitiatingHeapOccupancyPercent=70 -XX:MaxRAMPercentage=90.0 -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=/tmp"

## @param serviceapi.queues define the number for the queues
## @param serviceapi.queues.totalNumber define the total number of queues
## @param serviceapi.queues.perPodNumber define the number of queues per pod
## How to calculate: @param serviceapi.queues.perPodNumber = @param serviceapi.queues.totalNumber / @ param serviceapi.replicaCount
Copy link

@tomikonio tomikonio Aug 13, 2024

Choose a reason for hiding this comment

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

Hi,
Why has this been removed? Is it recommended for the total queue number to be equal to the per-pod number now? (in case we have more than one API pod)

raikbitters added a commit that referenced this pull request Aug 16, 2024
* Update chart version and image versions

* Allow parametrization of DB password key

* Global Values & Custom RBAC  (#401)

* Image registry as global parameter #389

* Improper k8s RBAC API Groups #373

* Update RabbitMQ plugin set (#403)

* Update AMQP settings in api-deployment.yaml

* Add pre-upgrade hook  (#407)

* Update analyzer images (#410)

---------

Co-authored-by: piotr-smietana-kaseya <piotr.smietana@kaseya.com>
Co-authored-by: Hleb Kanonik <hleb_kanonik@epam.com>
Co-authored-by: millerick <michael.millerick@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants