Skip to content

Commit

Permalink
Fix issues caught in review
Browse files Browse the repository at this point in the history
  • Loading branch information
silvestre committed Oct 31, 2023
1 parent 000adca commit c11e8d8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion api/scaling-history-api.openapi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ paths:
schema:
type: string
enum: ["asc", "desc"]
deprecated: true
deprecated: true
example: order=desc
- name: page
in: query
Expand Down
4 changes: 2 additions & 2 deletions src/autoscaler/api/publicapiserver/scaling_history_handler.go
Original file line number Diff line number Diff line change
Expand Up @@ -46,9 +46,9 @@ func NewScalingHistoryHandler(logger lager.Logger, conf *config.Config) (*Scalin

func (h *ScalingHistoryHandler) NewError(_ context.Context, _ error) *scalinghistory.ErrorResponseStatusCode {
result := &scalinghistory.ErrorResponseStatusCode{}
result.SetStatusCode(500)
result.SetStatusCode(http.StatusInternalServerError)
result.SetResponse(scalinghistory.ErrorResponse{
Code: scalinghistory.NewOptString(http.StatusText(500)),
Code: scalinghistory.NewOptString(http.StatusText(http.StatusInternalServerError)),
Message: scalinghistory.NewOptString("Error retrieving scaling history from scaling engine"),
})
return result
Expand Down

0 comments on commit c11e8d8

Please sign in to comment.