Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(aws): Fixing bugs related to clone CX when instance types are incompatible with image/region #9901

Merged
merged 4 commits into from Nov 8, 2022

Conversation

pdk27
Copy link
Contributor

@pdk27 pdk27 commented Nov 3, 2022

Changes

  • Tracking previously selected multiple instance types and handling mode change
  • Displaying warnings for clear indication of removed instance types due to incompatibility in image / region
  • Validating form on image change as an image change can unselect instance types
  • Populating instanceType for multiple instance types case (when overrides are set) for backwards compatibility

spinnaker/spinnaker#6734

Bug

NoValidationErrorUntilClick.mov

Fix

Fix.mov

…ompatible with image/region

Displaying warnings when incompatible instance types are removed

spinnaker/spinnaker#6734
@dbyron-sf
Copy link
Contributor

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Nov 7, 2022

update

✅ Branch has been successfully updated

@dbyron-sf dbyron-sf added the ready to merge Reviewed and ready for merge label Nov 8, 2022
@mergify mergify bot added the auto merged Merged automatically by a bot label Nov 8, 2022
@mergify mergify bot merged commit d7290c4 into spinnaker:master Nov 8, 2022
@dbyron-sf
Copy link
Contributor

@Mergifyio backport release-1.29.x

@mergify
Copy link
Contributor

mergify bot commented Nov 8, 2022

backport release-1.29.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Nov 8, 2022
…ompatible with image/region (#9901)

* fix(aws): Fixing bugs related to clone UX when instance types are incompatible with image/region

Displaying warnings when incompatible instance types are removed

spinnaker/spinnaker#6734

* fix(aws): PR feedback

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit d7290c4)
mergify bot added a commit that referenced this pull request Nov 8, 2022
…ompatible with image/region (#9901) (#9907)

* fix(aws): Fixing bugs related to clone UX when instance types are incompatible with image/region

Displaying warnings when incompatible instance types are removed

spinnaker/spinnaker#6734

* fix(aws): PR feedback

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit d7290c4)

Co-authored-by: Prathibha Datta Kumar <kprathib@amazon.com>
@dogonthehorizon
Copy link
Contributor

Are there any concerns with also backporting this to 1.28?

@dogonthehorizon
Copy link
Contributor

@Mergifyio backport release-1.28.x

@mergify
Copy link
Contributor

mergify bot commented Apr 5, 2023

backport release-1.28.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Apr 5, 2023
…ompatible with image/region (#9901)

* fix(aws): Fixing bugs related to clone UX when instance types are incompatible with image/region

Displaying warnings when incompatible instance types are removed

spinnaker/spinnaker#6734

* fix(aws): PR feedback

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit d7290c4)

# Conflicts:
#	packages/amazon/src/serverGroup/configure/serverGroupConfiguration.service.ts
#	packages/amazon/src/serverGroup/configure/wizard/instanceType/advancedMode/AmazonInstanceTypeInfoRenderer.tsx
link108 pushed a commit to armory-io/deck that referenced this pull request Apr 17, 2023
…ompatible with image/region (spinnaker#9901)

* fix(aws): Fixing bugs related to clone UX when instance types are incompatible with image/region

Displaying warnings when incompatible instance types are removed

spinnaker/spinnaker#6734

* fix(aws): PR feedback

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
link108 pushed a commit that referenced this pull request Apr 17, 2023
…ompatible with image/region (#9901)

* fix(aws): Fixing bugs related to clone UX when instance types are incompatible with image/region

Displaying warnings when incompatible instance types are removed

spinnaker/spinnaker#6734

* fix(aws): PR feedback

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
link108 added a commit that referenced this pull request Apr 18, 2023
#9980)

Co-authored-by: Cameron Motevasselani <cmotevasselani@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Prathibha Datta Kumar <kprathib@amazon.com>
fix(aws): Fixing bugs related to clone CX when instance types are incompatible with image/region (#9901)
link108 added a commit that referenced this pull request Apr 20, 2023
…ompatible with image/region (backport #9901) (#9975)

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Cameron Motevasselani <cmotevasselani@gmail.com>
Co-authored-by: Prathibha Datta Kumar <kprathib@amazon.com>
fix(aws): Fixing bugs related to clone CX when instance types are incompatible with image/region (#9901)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Reviewed and ready for merge
Projects
None yet
4 participants