Skip to content

Conversation

@ffjlabo
Copy link
Member

@ffjlabo ffjlabo commented Nov 14, 2023

What this PR does / why we need it:

Some annotations for k8s resources are not described in official docs. So I added docs.

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

  • How are users affected by this change:
  • Is this breaking change:
  • How to migrate (if breaking change):

Signed-off-by: Yoshiki Fujikane <[email protected]>
@codecov
Copy link

codecov bot commented Nov 14, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (99509fd) 30.81% compared to head (3d84794) 30.82%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4673   +/-   ##
=======================================
  Coverage   30.81%   30.82%           
=======================================
  Files         221      221           
  Lines       25935    25935           
=======================================
+ Hits         7993     7995    +2     
+ Misses      17292    17290    -2     
  Partials      650      650           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@khanhtc1202 khanhtc1202 left a comment

Choose a reason for hiding this comment

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

Good catch

@ffjlabo ffjlabo merged commit 318cca9 into master Nov 14, 2023
@ffjlabo ffjlabo deleted the add-docs-for-annotations branch November 14, 2023 09:10
moko-poi pushed a commit to moko-poi/pipecd that referenced this pull request Nov 15, 2023
Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: moko-poi <[email protected]>
khanhtc1202 added a commit that referenced this pull request Nov 28, 2023
* feat(ECS): enable selection of listener rules

Signed-off-by: moko-poi <[email protected]>

* Update pkg/app/piped/platformprovider/ecs/client.go

Signed-off-by: moko-poi <[email protected]>

* Make actions plan preview handle timeout option (#4648)

* Format and make tests pass in tool/actions-plan-preview

Signed-off-by: mi11km <[email protected]>

* Use --piped-handle-timeout to make the timeout arg activate

Signed-off-by: mi11km <[email protected]>

* Add tests to model/notificationevent (#4631)

* Add tests to model/notificationevent

Signed-off-by: Kenta Kozuka <[email protected]>

* Add Parallel()

Signed-off-by: Kenta Kozuka <[email protected]>

---------

Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: mi11km <[email protected]>

* add piped-handle-timeout(input args)

Signed-off-by: mi11km <[email protected]>

---------

Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* Increase limit of stale action operations (#4671)

Signed-off-by: moko-poi <[email protected]>

* Add docs for k8s annotations (#4673)

Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* feat: add modifylisner and modifyrule

Signed-off-by: moko-poi <[email protected]>

* fix: ModifyRules

Signed-off-by: moko-poi <[email protected]>

* fix: rollback

Signed-off-by: moko-poi <[email protected]>

* fix: routing

Signed-off-by: moko-poi <[email protected]>

* fix: action.Type == elbtypes.ActionTypeEnumForward

Signed-off-by: moko-poi <[email protected]>

* fix: Update ModifyListeners and ModifyRules for Selective Action Modification

Signed-off-by: moko-poi <[email protected]>

---------

Signed-off-by: moko-poi <[email protected]>
Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: Masafumi Ikeyama <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>
khanhtc1202 added a commit that referenced this pull request Nov 28, 2023
* feat(ECS): enable selection of listener rules

Signed-off-by: moko-poi <[email protected]>

* Update pkg/app/piped/platformprovider/ecs/client.go

Signed-off-by: moko-poi <[email protected]>

* Make actions plan preview handle timeout option (#4648)

* Format and make tests pass in tool/actions-plan-preview

Signed-off-by: mi11km <[email protected]>

* Use --piped-handle-timeout to make the timeout arg activate

Signed-off-by: mi11km <[email protected]>

* Add tests to model/notificationevent (#4631)

* Add tests to model/notificationevent

Signed-off-by: Kenta Kozuka <[email protected]>

* Add Parallel()

Signed-off-by: Kenta Kozuka <[email protected]>

---------

Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: mi11km <[email protected]>

* add piped-handle-timeout(input args)

Signed-off-by: mi11km <[email protected]>

---------

Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* Increase limit of stale action operations (#4671)

Signed-off-by: moko-poi <[email protected]>

* Add docs for k8s annotations (#4673)

Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* feat: add modifylisner and modifyrule

Signed-off-by: moko-poi <[email protected]>

* fix: ModifyRules

Signed-off-by: moko-poi <[email protected]>

* fix: rollback

Signed-off-by: moko-poi <[email protected]>

* fix: routing

Signed-off-by: moko-poi <[email protected]>

* fix: action.Type == elbtypes.ActionTypeEnumForward

Signed-off-by: moko-poi <[email protected]>

* fix: Update ModifyListeners and ModifyRules for Selective Action Modification

Signed-off-by: moko-poi <[email protected]>

---------

Signed-off-by: moko-poi <[email protected]>
Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: Masafumi Ikeyama <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>
khanhtc1202 added a commit that referenced this pull request Nov 28, 2023
* feat(ECS): enable selection of listener rules (#4668)

* feat(ECS): enable selection of listener rules

Signed-off-by: moko-poi <[email protected]>

* Update pkg/app/piped/platformprovider/ecs/client.go

Signed-off-by: moko-poi <[email protected]>

* Make actions plan preview handle timeout option (#4648)

* Format and make tests pass in tool/actions-plan-preview

Signed-off-by: mi11km <[email protected]>

* Use --piped-handle-timeout to make the timeout arg activate

Signed-off-by: mi11km <[email protected]>

* Add tests to model/notificationevent (#4631)

* Add tests to model/notificationevent

Signed-off-by: Kenta Kozuka <[email protected]>

* Add Parallel()

Signed-off-by: Kenta Kozuka <[email protected]>

---------

Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: mi11km <[email protected]>

* add piped-handle-timeout(input args)

Signed-off-by: mi11km <[email protected]>

---------

Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* Increase limit of stale action operations (#4671)

Signed-off-by: moko-poi <[email protected]>

* Add docs for k8s annotations (#4673)

Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* feat: add modifylisner and modifyrule

Signed-off-by: moko-poi <[email protected]>

* fix: ModifyRules

Signed-off-by: moko-poi <[email protected]>

* fix: rollback

Signed-off-by: moko-poi <[email protected]>

* fix: routing

Signed-off-by: moko-poi <[email protected]>

* fix: action.Type == elbtypes.ActionTypeEnumForward

Signed-off-by: moko-poi <[email protected]>

* fix: Update ModifyListeners and ModifyRules for Selective Action Modification

Signed-off-by: moko-poi <[email protected]>

---------

Signed-off-by: moko-poi <[email protected]>
Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: Masafumi Ikeyama <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>

* Remove ModifyRules interface (#4688)

---------

Signed-off-by: moko-poi <[email protected]>
Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Shun Takahashi <[email protected]>
Co-authored-by: Masafumi Ikeyama <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>
@github-actions github-actions bot mentioned this pull request Dec 1, 2023
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Dec 17, 2023
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Dec 17, 2023
* feat(ECS): enable selection of listener rules

Signed-off-by: moko-poi <[email protected]>

* Update pkg/app/piped/platformprovider/ecs/client.go

Signed-off-by: moko-poi <[email protected]>

* Make actions plan preview handle timeout option (pipe-cd#4648)

* Format and make tests pass in tool/actions-plan-preview

Signed-off-by: mi11km <[email protected]>

* Use --piped-handle-timeout to make the timeout arg activate

Signed-off-by: mi11km <[email protected]>

* Add tests to model/notificationevent (pipe-cd#4631)

* Add tests to model/notificationevent

Signed-off-by: Kenta Kozuka <[email protected]>

* Add Parallel()

Signed-off-by: Kenta Kozuka <[email protected]>

---------

Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: mi11km <[email protected]>

* add piped-handle-timeout(input args)

Signed-off-by: mi11km <[email protected]>

---------

Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* Increase limit of stale action operations (pipe-cd#4671)

Signed-off-by: moko-poi <[email protected]>

* Add docs for k8s annotations (pipe-cd#4673)

Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* feat: add modifylisner and modifyrule

Signed-off-by: moko-poi <[email protected]>

* fix: ModifyRules

Signed-off-by: moko-poi <[email protected]>

* fix: rollback

Signed-off-by: moko-poi <[email protected]>

* fix: routing

Signed-off-by: moko-poi <[email protected]>

* fix: action.Type == elbtypes.ActionTypeEnumForward

Signed-off-by: moko-poi <[email protected]>

* fix: Update ModifyListeners and ModifyRules for Selective Action Modification

Signed-off-by: moko-poi <[email protected]>

---------

Signed-off-by: moko-poi <[email protected]>
Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: Masafumi Ikeyama <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Dec 17, 2023
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Dec 17, 2023
* feat(ECS): enable selection of listener rules

Signed-off-by: moko-poi <[email protected]>

* Update pkg/app/piped/platformprovider/ecs/client.go

Signed-off-by: moko-poi <[email protected]>

* Make actions plan preview handle timeout option (pipe-cd#4648)

* Format and make tests pass in tool/actions-plan-preview

Signed-off-by: mi11km <[email protected]>

* Use --piped-handle-timeout to make the timeout arg activate

Signed-off-by: mi11km <[email protected]>

* Add tests to model/notificationevent (pipe-cd#4631)

* Add tests to model/notificationevent

Signed-off-by: Kenta Kozuka <[email protected]>

* Add Parallel()

Signed-off-by: Kenta Kozuka <[email protected]>

---------

Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: mi11km <[email protected]>

* add piped-handle-timeout(input args)

Signed-off-by: mi11km <[email protected]>

---------

Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* Increase limit of stale action operations (pipe-cd#4671)

Signed-off-by: moko-poi <[email protected]>

* Add docs for k8s annotations (pipe-cd#4673)

Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* feat: add modifylisner and modifyrule

Signed-off-by: moko-poi <[email protected]>

* fix: ModifyRules

Signed-off-by: moko-poi <[email protected]>

* fix: rollback

Signed-off-by: moko-poi <[email protected]>

* fix: routing

Signed-off-by: moko-poi <[email protected]>

* fix: action.Type == elbtypes.ActionTypeEnumForward

Signed-off-by: moko-poi <[email protected]>

* fix: Update ModifyListeners and ModifyRules for Selective Action Modification

Signed-off-by: moko-poi <[email protected]>

---------

Signed-off-by: moko-poi <[email protected]>
Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: Masafumi Ikeyama <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Dec 17, 2023
Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: sZma5a <[email protected]>
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Dec 17, 2023
* feat(ECS): enable selection of listener rules

Signed-off-by: moko-poi <[email protected]>

* Update pkg/app/piped/platformprovider/ecs/client.go

Signed-off-by: moko-poi <[email protected]>

* Make actions plan preview handle timeout option (pipe-cd#4648)

* Format and make tests pass in tool/actions-plan-preview

Signed-off-by: mi11km <[email protected]>

* Use --piped-handle-timeout to make the timeout arg activate

Signed-off-by: mi11km <[email protected]>

* Add tests to model/notificationevent (pipe-cd#4631)

* Add tests to model/notificationevent

Signed-off-by: Kenta Kozuka <[email protected]>

* Add Parallel()

Signed-off-by: Kenta Kozuka <[email protected]>

---------

Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: mi11km <[email protected]>

* add piped-handle-timeout(input args)

Signed-off-by: mi11km <[email protected]>

---------

Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* Increase limit of stale action operations (pipe-cd#4671)

Signed-off-by: moko-poi <[email protected]>

* Add docs for k8s annotations (pipe-cd#4673)

Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* feat: add modifylisner and modifyrule

Signed-off-by: moko-poi <[email protected]>

* fix: ModifyRules

Signed-off-by: moko-poi <[email protected]>

* fix: rollback

Signed-off-by: moko-poi <[email protected]>

* fix: routing

Signed-off-by: moko-poi <[email protected]>

* fix: action.Type == elbtypes.ActionTypeEnumForward

Signed-off-by: moko-poi <[email protected]>

* fix: Update ModifyListeners and ModifyRules for Selective Action Modification

Signed-off-by: moko-poi <[email protected]>

---------

Signed-off-by: moko-poi <[email protected]>
Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: Masafumi Ikeyama <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: sZma5a <[email protected]>
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Dec 17, 2023
* feat(ECS): enable selection of listener rules

Signed-off-by: moko-poi <[email protected]>

* Update pkg/app/piped/platformprovider/ecs/client.go

Signed-off-by: moko-poi <[email protected]>

* Make actions plan preview handle timeout option (pipe-cd#4648)

* Format and make tests pass in tool/actions-plan-preview

Signed-off-by: mi11km <[email protected]>

* Use --piped-handle-timeout to make the timeout arg activate

Signed-off-by: mi11km <[email protected]>

* Add tests to model/notificationevent (pipe-cd#4631)

* Add tests to model/notificationevent

Signed-off-by: Kenta Kozuka <[email protected]>

* Add Parallel()

Signed-off-by: Kenta Kozuka <[email protected]>

---------

Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: mi11km <[email protected]>

* add piped-handle-timeout(input args)

Signed-off-by: mi11km <[email protected]>

---------

Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* Increase limit of stale action operations (pipe-cd#4671)

Signed-off-by: moko-poi <[email protected]>

* Add docs for k8s annotations (pipe-cd#4673)

Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* feat: add modifylisner and modifyrule

Signed-off-by: moko-poi <[email protected]>

* fix: ModifyRules

Signed-off-by: moko-poi <[email protected]>

* fix: rollback

Signed-off-by: moko-poi <[email protected]>

* fix: routing

Signed-off-by: moko-poi <[email protected]>

* fix: action.Type == elbtypes.ActionTypeEnumForward

Signed-off-by: moko-poi <[email protected]>

* fix: Update ModifyListeners and ModifyRules for Selective Action Modification

Signed-off-by: moko-poi <[email protected]>

---------

Signed-off-by: moko-poi <[email protected]>
Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: Masafumi Ikeyama <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: sZma5a <[email protected]>
@github-actions github-actions bot mentioned this pull request Feb 6, 2024
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Feb 12, 2024
Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: 鈴木 優耀 <[email protected]>
sZma5a pushed a commit to sZma5a/pipecd that referenced this pull request Feb 12, 2024
* feat(ECS): enable selection of listener rules

Signed-off-by: moko-poi <[email protected]>

* Update pkg/app/piped/platformprovider/ecs/client.go

Signed-off-by: moko-poi <[email protected]>

* Make actions plan preview handle timeout option (pipe-cd#4648)

* Format and make tests pass in tool/actions-plan-preview

Signed-off-by: mi11km <[email protected]>

* Use --piped-handle-timeout to make the timeout arg activate

Signed-off-by: mi11km <[email protected]>

* Add tests to model/notificationevent (pipe-cd#4631)

* Add tests to model/notificationevent

Signed-off-by: Kenta Kozuka <[email protected]>

* Add Parallel()

Signed-off-by: Kenta Kozuka <[email protected]>

---------

Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: mi11km <[email protected]>

* add piped-handle-timeout(input args)

Signed-off-by: mi11km <[email protected]>

---------

Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* Increase limit of stale action operations (pipe-cd#4671)

Signed-off-by: moko-poi <[email protected]>

* Add docs for k8s annotations (pipe-cd#4673)

Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: moko-poi <[email protected]>

* feat: add modifylisner and modifyrule

Signed-off-by: moko-poi <[email protected]>

* fix: ModifyRules

Signed-off-by: moko-poi <[email protected]>

* fix: rollback

Signed-off-by: moko-poi <[email protected]>

* fix: routing

Signed-off-by: moko-poi <[email protected]>

* fix: action.Type == elbtypes.ActionTypeEnumForward

Signed-off-by: moko-poi <[email protected]>

* fix: Update ModifyListeners and ModifyRules for Selective Action Modification

Signed-off-by: moko-poi <[email protected]>

---------

Signed-off-by: moko-poi <[email protected]>
Signed-off-by: mi11km <[email protected]>
Signed-off-by: Kenta Kozuka <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: Masafumi Ikeyama <[email protected]>
Co-authored-by: Kenta Kozuka <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: 鈴木 優耀 <[email protected]>
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