This message was deleted.
# generals
sparse-intern-71089
08/22/2019, 10:18 PMThis message was deleted.
c
creamy-potato-29402
08/23/2019, 12:38 AMcc @white-balloon-205— @cool-egg-852 I believe we accurately report whether the underlying cloud provider allows us to do an in-place update of any particular property.
c
cool-egg-852
08/23/2019, 1:30 AMI can change it in the console without it being recreated
cool-egg-852
08/23/2019, 2:58 PMAre you saying their API doesn’t allow for what you can do in the console? If so, that’s quite stupid of them.
c
creamy-potato-29402
08/23/2019, 5:07 PM@cool-egg-852 I’m not an expert—just, generally, my understanding is that a property is not marked for replacement unless we have to replace the resource. perhaps @stocky-spoon-28903 @broad-dog-22463 have more information?
w
white-balloon-205
08/23/2019, 5:36 PM
Which resource is this specifically? There are frequently either cloud provider issues or upstream provider issues that lead to overly conservative treatments of replacement requirements - but would have to look into a specific resource to be sure.
c
cool-egg-852
08/23/2019, 5:39 PMThis is a gcp.sql.DatabaseInstance
cool-egg-852
08/23/2019, 5:40 PMSpecifically postgres, as that is the only DB that
availabilityTypeb
broad-dog-22463
08/23/2019, 6:22 PMCan you show me what you ate chanomf?
broad-dog-22463
08/23/2019, 6:22 PMChanging*
broad-dog-22463
08/23/2019, 6:22 PM(Sorry autocorrect!)
c
cool-egg-852
08/23/2019, 7:14 PMAll I did was add
availabilityType: config.environment == 'production' ? 'REGIONAL' : 'ZONAL'DatabaseInstancecool-egg-852
08/23/2019, 7:15 PMAnd of course the value was production, so
REGIONALcool-egg-852
08/23/2019, 7:31 PMIt’s not trying to replace the DatabaseInstance, at least not anymore. Instead, it’s trying to replace the user and the database
cool-egg-852
08/23/2019, 7:31 PMThis doesn’t make any sense. The instance isn’t being recreated, but at minimum the previous is saying it will do a create-replacement on the user and Database.
cool-egg-852
08/23/2019, 7:51 PMI’m testing this out. I’m beginning to think this is based upon the limitation of the preview. The preview seems to assume changes are required that are not actually required when executing.
cool-egg-852
08/23/2019, 7:52 PMThis same thing scared me a few minutes ago when updating a configMap. It caused the deployment to show it was going to be replaced. It was only updated in the end.
cool-egg-852
08/23/2019, 7:59 PMYep, that’s exactly what happened.
cool-egg-852
08/23/2019, 7:59 PMIs there no way to fix this? The preview should not be this off from reality.
cool-egg-852
08/28/2019, 2:49 PMI’m curious why this has gotten worse rather than better. I never experienced this previously where the diff was so off.