Skip to content

Commit

Permalink
Allow rate-limit of changes on cluster (#147)
Browse files Browse the repository at this point in the history
* Add initial draft to allow limit number of changes on cluster

* Update lint

* Simplify way to measure progress of system rollouts by emiting k8s events

* Applying @crgisch suggestions, thanks
  • Loading branch information
wpjunior authored Mar 20, 2024
1 parent 6e3ae69 commit dbec591
Show file tree
Hide file tree
Showing 9 changed files with 767 additions and 102 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
go-version: "1.21"
- uses: golangci/golangci-lint-action@v3
with:
version: v1.55
version: v1.56
- run: make test

integration:
Expand Down
5 changes: 4 additions & 1 deletion api/v1alpha1/rpaasinstance_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -340,7 +340,10 @@ type TLSSessionTicket struct {

// RpaasInstanceStatus defines the observed state of RpaasInstance
type RpaasInstanceStatus struct {
// Revision hash calculated for the current spec.
//Revision hash calculated for the current spec of rpaasinstance
RevisionHash string `json:"revisionHash,omitempty"`

// Revision hash calculated for the current spec of nginx.
WantedNginxRevisionHash string `json:"wantedNginxRevisionHash,omitempty"`

// The revision hash observed by the controller in the nginx object.
Expand Down
5 changes: 4 additions & 1 deletion config/crd/bases/extensions.tsuru.io_rpaasinstances.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6476,8 +6476,11 @@ spec:
podSelector:
description: PodSelector is the NGINX's pod label selector.
type: string
revisionHash:
description: Revision hash calculated for the current spec of rpaasinstance
type: string
wantedNginxRevisionHash:
description: Revision hash calculated for the current spec.
description: Revision hash calculated for the current spec of nginx.
type: string
required:
- nginxUpdated
Expand Down
Loading

0 comments on commit dbec591

Please sign in to comment.