Skip to content
This repository has been archived by the owner on Mar 11, 2021. It is now read-only.

add capacity_exhausted to cluster table #29

Merged
merged 3 commits into from
Nov 22, 2018

Conversation

dipak-pawar
Copy link
Contributor

Changes proposed in this PR

  • add capacity_exhausted column to cluster table
  • Modifies test accordingly

Copy link
Contributor

@alexeykazakov alexeykazakov left a comment

Choose a reason for hiding this comment

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

Any tests adjustments for the new flag?

@codecov
Copy link

codecov bot commented Nov 22, 2018

Codecov Report

Merging #29 into master will decrease coverage by 0.21%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   75.83%   75.62%   -0.22%     
==========================================
  Files          12       12              
  Lines         716      718       +2     
==========================================
  Hits          543      543              
- Misses        142      143       +1     
- Partials       31       32       +1
Impacted Files Coverage Δ
cluster/repository/cluster.go 69.81% <ø> (ø) ⬆️
controller/clusters.go 100% <100%> (ø) ⬆️
cluster/service/cluster.go 68.67% <100%> (-2.06%) ⬇️
migration/migration.go 84.61% <100%> (+1.28%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23f2d0c...651123b. Read the comment docs.

Copy link
Contributor

@xcoulon xcoulon left a comment

Choose a reason for hiding this comment

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

just a minor comment, otherwise, it looks good to me 👍

@@ -0,0 +1,2 @@
-- Store capacity_exhausted for cluster
ALTER TABLE cluster ADD COLUMN capacity_exhausted boolean;
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe add DEFAULT false to the column definition, for existing clusters in the DB?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

addressed in 651123b

@dipak-pawar dipak-pawar merged commit a9f2727 into fabric8-services:master Nov 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants