Skip to content

Conversation

@icezohu
Copy link
Collaborator

@icezohu icezohu commented Oct 10, 2024

No description provided.

@pamanseau
Copy link

I get this error:
subquery-coordinator-558857d47d-nlbdw.log

@icezohu
Copy link
Collaborator Author

icezohu commented Nov 22, 2024

I get this error: subquery-coordinator-558857d47d-nlbdw.log

Did you alter database tables like removing some?

@pamanseau
Copy link

pamanseau commented Nov 22, 2024

I get this error: subquery-coordinator-558857d47d-nlbdw.log

Did you alter database tables like removing some?

I just deleted the coordinator DB and created a new one.

here are the logs:
subquery-coordinator-784d7465d9-p72vt.log

Still trying to connect to the docker.sock

2024-11-22T09:25:59.479Z <�[95mCoordinatorMetricsService�[39m> �[31mERROR�[39m failed to push service versions 
Error: connect ENOENT /var/run/docker.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1278:16)

@icezohu
Copy link
Collaborator Author

icezohu commented Nov 22, 2024

I get this error: subquery-coordinator-558857d47d-nlbdw.log

Did you alter database tables like removing some?

I just deleted the coordinator DB and created a new one.

here are the logs: subquery-coordinator-784d7465d9-p72vt.log

Still trying to connect to the docker.sock

2024-11-22T09:25:59.479Z <�[95mCoordinatorMetricsService�[39m> �[31mERROR�[39m failed to push service versions 
Error: connect ENOENT /var/run/docker.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1278:16)

It's just some annoying logs for docker.sock.

Please make sure to delete the migration table and coordinator will try to rebuild the db structure.

@pamanseau
Copy link

@icezohu
I deleted the table here is the log

subquery-coordinator-784d7465d9-ss7t8.log

@pamanseau
Copy link

@icezohu Do you have time to look into this?
Thank you

# Conflicts:
#	apps/indexer-coordinator/package.json
@pamanseau
Copy link

@icezohu Can we continue this PR?

@pamanseau
Copy link

Tested with argv --host-env=k8s and it's working.

Thanks

@pamanseau
Copy link

I will be able to complete the helm chart for issue #422 and create a PR

@cool-firer cool-firer merged commit 063eda5 into main Mar 10, 2025
2 checks passed
@icezohu icezohu deleted the k8s-compatibility branch March 24, 2025 08:22
@Faizalhidayat
Copy link

So how about reward ? @icezohu

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.

5 participants