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

[v2] improve the log messages #1565

Closed
postmaxin opened this issue Oct 26, 2020 · 7 comments · Fixed by #1637
Closed

[v2] improve the log messages #1565

postmaxin opened this issue Oct 26, 2020 · 7 comments · Fixed by #1637

Comments

@postmaxin
Copy link

Seeing a lot of stuff like this;

{"level":"error","ts":1603747129.250157,"logger":"controller","msg":"Reconciler error","controller":"ingress","name":"rex-ingress-external","namespace":"","error":"cannot find at least two subnets from different Availability Zones, discovered subnetIDs: []"}
{"level":"error","ts":1603747155.3905587,"logger":"controller","msg":"Reconciler error","controller":"ingress","name":"rex-ingress-dev","namespace":"","error":"Service \"qualia-sync-cron-service\" not found"}
@M00nF1sh
Copy link
Collaborator

@postmaxin
It's expected. I assume you are using rex-ingress-external and rex-ingress-dev as IngressGroups. these are encoded as a types.NamespacedName without namespace.

Do you need help on the actual error?

  1. the first one should be either u didn't tag the subnets correctly or u didn't set the '--cluster-name` flag to controller correctly
  2. the second one is the service referenced by one Ingress in IngressGroup rex-ingress-dev is not found

@postmaxin
Copy link
Author

So for #1, that totally makes sense, and I'm fine.

For #2 -- it was looking for the service in a namespace, correct? How do we tell what namespace it was looking in?

@M00nF1sh
Copy link
Collaborator

I agree we should improve the logs for #2. Before that, you can check all ingresses with group.name rex-ingress-dev :D.

@postmaxin
Copy link
Author

Oh boy! :D

$ kubectl get ing -A | grep rex-ingress-dev | wc -l
   158

Ok, I'll do that for now

@M00nF1sh
Copy link
Collaborator

M00nF1sh commented Oct 27, 2020

@postmaxin
We'll track the fix in v2.0.1

@M00nF1sh M00nF1sh reopened this Oct 27, 2020
@M00nF1sh M00nF1sh changed the title [v2] Null namespaces in logs [v2] optimize the log messages Oct 27, 2020
@M00nF1sh M00nF1sh changed the title [v2] optimize the log messages [v2] improve the log messages Oct 27, 2020
@weberc2-tempus
Copy link

the first one should be either u didn't tag the subnets correctly or u didn't set the '--cluster-name` flag to controller correctly

How can we debug the --cluster-name flag setting?

@M00nF1sh
Copy link
Collaborator

@weberc2-tempus the --cluster-name setting is a flag to the controller, we currently didn't print it out in the logs. maybe in future versions, we can echo them.

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 a pull request may close this issue.

3 participants