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

feat(node): add default akash snapshot provider and remove duplicate port definition #293

Merged
merged 2 commits into from
Sep 26, 2024

Conversation

andy108369
Copy link
Collaborator

@andy108369 andy108369 commented Sep 26, 2024

two commits in one PR, please see Commits tab

fixes akash-network/support#287

fixes akash-network/support#287
to tackle these warnings:

$ helm install akash-node ./charts/akash-node -n akash-services
W0926 11:56:36.874077  117617 warnings.go:70] spec.template.spec.containers[0].ports[0]: duplicate port definition with spec.template.spec.initContainers[0].ports[0]
W0926 11:56:36.874089  117617 warnings.go:70] spec.template.spec.containers[0].ports[1]: duplicate port definition with spec.template.spec.initContainers[0].ports[1]
W0926 11:56:36.874094  117617 warnings.go:70] spec.template.spec.containers[0].ports[2]: duplicate port definition with spec.template.spec.initContainers[0].ports[2]
W0926 11:56:36.874096  117617 warnings.go:70] spec.template.spec.containers[0].ports[3]: duplicate port definition with spec.template.spec.initContainers[0].ports[3]
Copy link
Contributor

@chainzero chainzero left a comment

Choose a reason for hiding this comment

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

Reviewed updates and all looks good

chainzero

This comment was marked as duplicate.

@chainzero chainzero merged commit 10b74c5 into main Sep 26, 2024
2 checks passed
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.

2 participants