Skip to content

Conversation

@joyyir
Copy link
Contributor

@joyyir joyyir commented Jan 3, 2024

feat: Capable of overriding the configurations of the nginx http lua_shared_dict

The Lua shared dictionary prometheus-metrics is initially configured with a default size of 10m as specified in the config-default.yaml file. However, due to the growing number of routes, the dictionary size has been exceeded. Hence, customization of this configuration would prove advantageous.

@joyyir joyyir changed the title feat: Capable of overriding the configurations of the nginx http lua_shared_dict (2.x.x) feat: Capable of overriding the configurations of the nginx http lua_shared_dict Jan 3, 2024
@kworkbee
Copy link
Contributor

kworkbee commented Apr 16, 2024

Hello, @Revolyssup @shreemaan-abhishek @tao12345666333 @AlinsRan @lingsamuel can you please take a look at this?

@kworkbee
Copy link
Contributor

I'd like to reflect the changes in the company's operating environment, but does it take time from Merge to Release?

@kworkbee
Copy link
Contributor

Hello, @Revolyssup @shreemaan-abhishek @tao12345666333 @AlinsRan @lingsamuel
Who has permission to merge for this PR?

@shreemaan-abhishek shreemaan-abhishek merged commit 9660fe3 into apache:dev Aug 4, 2024
@kworkbee
Copy link
Contributor

kworkbee commented Mar 24, 2025

Hello @shreemaan-abhishek / @Revolyssup ,
The target branch is set incorrectly and it seems that the feature is not currently reflected in the latest Chart. Could you please take a look into it?

@swapnil-omni
Copy link

Any idea when can we get this in latest chart?

@kworkbee
Copy link
Contributor

kworkbee commented May 7, 2025

Hi, @moonming Can you please look at this?

@moonming
Copy link
Member

moonming commented May 8, 2025

Hi, @moonming Can you please look at this?

Will fix it, thanks @kworkbee

@Baoyuantop
Copy link
Contributor

Hi @kworkbee, #822 is working on this issue and has been submitted to the correct branch. This issue will be fixed after it is merged.

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.

8 participants