Skip to content

Conversation

@terassyi
Copy link
Contributor

@terassyi terassyi commented Mar 3, 2025

Signed-off-by: terashima tomoya-terashima@cybozu.co.jp

@terassyi terassyi self-assigned this Mar 3, 2025
@terassyi terassyi force-pushed the fix-controller-name branch from 5b1574d to b9999ab Compare March 3, 2025 07:54
@terassyi terassyi requested a review from chez-shanpu March 3, 2025 07:59
@terassyi terassyi marked this pull request as ready for review March 3, 2025 07:59
Copy link
Contributor

@chez-shanpu chez-shanpu left a comment

Choose a reason for hiding this comment

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

Thank you for your works.

I left a small comment.
And it seems there are still some "coil-controller" exists. e.g. v2/dashboard/coil.json, docs/usage.md
Could you check if they have to be updated?

Comment on lines 2 to 3
- ../config/default
- ../config/pod/coil-router.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- ../config/default
- ../config/pod/coil-router.yaml
- ../config/default
- ../config/pod/coil-router.yaml

nit: Other yaml files don't have spaces.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

my editor added spaces automatically.
I fixed this.

Copy link
Contributor

Choose a reason for hiding this comment

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

They're still around 👀

@terassyi terassyi force-pushed the fix-controller-name branch from b9999ab to 4fb56f2 Compare March 3, 2025 09:31
@terassyi terassyi requested a review from chez-shanpu March 3, 2025 09:32
Signed-off-by: terashima <tomoya-terashima@cybozu.co.jp>
@terassyi terassyi force-pushed the fix-controller-name branch from 4fb56f2 to 57e1a19 Compare March 3, 2025 09:38
Copy link
Contributor

@chez-shanpu chez-shanpu left a comment

Choose a reason for hiding this comment

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

I replied to your comment.
Others looks good to me :)

@terassyi terassyi requested a review from chez-shanpu March 4, 2025 01:06
Signed-off-by: terashima <tomoya-terashima@cybozu.co.jp>
@terassyi terassyi force-pushed the fix-controller-name branch from f6e75e6 to 4052da6 Compare March 4, 2025 01:52
Copy link
Contributor

@chez-shanpu chez-shanpu left a comment

Choose a reason for hiding this comment

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

LGTM

@terassyi terassyi merged commit 5f18da9 into main Mar 4, 2025
14 checks passed
@terassyi terassyi deleted the fix-controller-name branch March 4, 2025 02:12
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.

3 participants