search

linagora / tmail-backend

GNU Affero General Public License v3.0
36 stars 17 forks source link

Reduce a loooot the log volume of out apisix extensions #935

Closed chibenwa closed 5 months ago

chibenwa commented 5 months ago

Screenshot from 2024-02-23 22-08-54 Screenshot from 2024-02-23 22-08-42

Might be good for DEV

Questionable on PROD...

Cc @vttranlina

chibenwa commented 5 months ago

Goal: by default I should only have the access log, unless something goes really wrong.

chibenwa commented 5 months ago

image

200 logs per minute.

Where does this come from?

Is this necessary?

vttranlina commented 5 months ago
vttranlina commented 5 months ago

image

200 logs per minute.

Where does this come from?

Is this necessary?

It appears that the client tried to call the "not found"'s url (here is xxx/admin/xxx).

I couldn't find the /admin url in our apisix configuration file (docker-compose & helm emaily-staging-apps).

WDYT? @ducnm0711 @Arsnael

Arsnael commented 5 months ago

Would need to have a deeper look, no idea for now

vttranlina commented 5 months ago

The pr for tmail plugin is merged

Arsnael commented 5 months ago

Can you explain why you closed, then reopened the issue to then unassign yourself @vttranlina ? Is this finished or not?

chibenwa commented 5 months ago

Please update helm charts

vttranlina commented 5 months ago

Can you explain why you closed, then reopened the issue to then unassign yourself @vttranlina ? Is this finished or not?

It is not finished.

When I merged the code pr, the github-project-automation auto closed it But I think this task needs todo more. => re-opened manually

--- Why open?

As my note:

TODO, rebuild the image linagora/apisix:3.2.0-debian-javaplugin , and re-deploy on prod environment

I do not have permission to push dockerhub.

And Benoit's question at: https://github.com/linagora/tmail-backend/issues/935#issuecomment-1961997071, no idea for now. (I guess it comes from production setup)

ducnm0711 commented 5 months ago

https://github.com/apache/apisix-ingress-controller/issues/1855 Bug logged but no solution yet

ducnm0711 commented 5 months ago

Workaround - change log level from into --> error https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix-ingress-controller/values.yaml#L79

  ingress-controller:
    config:
      logLevel: "error"

image