Release v1.137.1 - staging → master

[jaalah-akamai]

Fixed:

• Unable to save non-US billing contact information without tax id (#11725)

[jaalah-akamai]

cc: @coliu-akamai Tags have been pushed already 👍

[coliu-akamai]

✅ confirmed specifications from prev pr

thank you

[cypress]

Cloud Manager E2E    Run #7293

Run Properties:   Failed #7293  •  b5db23fa52: Merge pull request #11729 from linode/staging

Project	Cloud Manager E2E
Branch Review	master
Run status	Failed #7293
Run duration	31m 24s
Commit	b5db23fa52: Merge pull request #11729 from linode/staging
Committer	Connie Liu
View all properties for this run ↗︎

---

Test results
Failures	1
Flaky	4
Pending	3
Skipped	0
Passing	520
View all changes introduced in this branch ↗︎

---

Tests for review

  cypress/e2e/core/cloudpulse/edit-system-alert.spec.ts • 1 failed test

View Output Video

Test		Artifacts
Integration Tests for Edit Alert > should correctly display and update the details of the alert in the edit alert page		Screenshots Video

  linodes/clone-linode.spec.ts • 1 flaky test

View Output Video

Test		Artifacts
clone linode > can clone a Linode from Linode details page		Screenshots Video

  stackscripts/smoke-community-stackscripts.spec.ts • 1 flaky test

View Output Video

Test		Artifacts
Community Stackscripts integration tests > pagination works with infinite scrolling		Screenshots Video

  parentChild/account-switching.spec.ts • 1 flaky test

View Output Video

Test		Artifacts
Parent/Child account switching > From Parent to Child > can switch from Parent account user to Proxy account user from Billing page		Screenshots Video

  placementGroups/delete-placement-groups.spec.ts • 1 flaky test

View Output Video

Test		Artifacts
Placement Group deletion > can delete without Linodes assigned when unexpected error show up and retry		Screenshots Video

[github-actions]

Coverage Report: ✅
Base Coverage: 79.98%
Current Coverage: 79.98%

[linode-gh-bot]

Cloud Manager UI test results

🔺 1 failing test on test run #2 ↗︎

❌ Failing	✅ Passing	↪️ Skipped	🕐 Duration
1 Failing	520 Passing	3 Skipped	110m 38s

Details

Failing Tests
	Spec	Test
❌	edit-system-alert.spec.ts	Integration Tests for Edit Alert » should correctly display and update the details of the alert in the edit alert page

Troubleshooting

Use this command to re-run the failing tests:

yarn cy:run -s "cypress/e2e/core/cloudpulse/edit-system-alert.spec.ts"

[jdamore-linode]

We can disregard the test failure here: it's been fixed in develop.

Just to clarify: I was under the impression this field is required for companies -- have we confirmed that this should be optional for all customers vs being optional only for individuals?

[jaalah-akamai]

@jdamore-linode The PM seemed pretty clear in my discussion that this is an optional field - Validation still occurs if there's a Tax Id and the agreement checkbox is still required to check, so I think we're safe here. I think leaving this as is until tomorrow will generate more support tickets

[coliu-akamai]

merging into master as per discussions!