Skip to content

TPT-4435: Upgrade test image linode/alpine3.19 to new version and regenerate fixture files#952

Merged
psnoch-akamai merged 2 commits into
linode:mainfrom
psnoch-akamai:TPT-4435-upgrade-linode-alpine3.19-to-linode-alpine3.23-for-linodego-tests
May 14, 2026
Merged

TPT-4435: Upgrade test image linode/alpine3.19 to new version and regenerate fixture files#952
psnoch-akamai merged 2 commits into
linode:mainfrom
psnoch-akamai:TPT-4435-upgrade-linode-alpine3.19-to-linode-alpine3.23-for-linodego-tests

Conversation

@psnoch-akamai
Copy link
Copy Markdown
Contributor

Upgrade test image linode/alpine3.19 to new version and regenerate fixture files

@psnoch-akamai psnoch-akamai force-pushed the TPT-4435-upgrade-linode-alpine3.19-to-linode-alpine3.23-for-linodego-tests branch from 80ec4d7 to 0f4a654 Compare May 6, 2026 14:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates integration tests to use a newer Alpine image (linode/alpine3.23) and regenerates the recorded VCR fixtures to reflect the updated API interactions.

Changes:

  • Updated instance rebuild integration tests to use linode/alpine3.23 instead of linode/alpine3.19.
  • Regenerated multiple integration VCR fixture YAMLs to match the new rebuild/image-list responses.
  • Fixture request URLs now point at https://api.devcloud.linode.com, which is likely to break fixture replay given the current test harness behavior.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
test/integration/instances_test.go Switches rebuild tests to use linode/alpine3.23.
test/integration/fixtures/TestInstance_Rebuild.yaml Regenerated rebuild cassette (now recorded against api.devcloud.linode.com).
test/integration/fixtures/TestInstance_RebuildWithEncryption.yaml Regenerated encrypted rebuild cassette (now recorded against api.devcloud.linode.com).
test/integration/fixtures/TestImages_List.yaml Regenerated images list cassette (now recorded against api.devcloud.linode.com).
test/integration/fixtures/ExampleListImages_all.yaml Regenerated example cassette for listing images (now recorded against api.devcloud.linode.com).
test/integration/fixtures/TestImageSharing_Suite.yaml Regenerated image-sharing suite cassette (now recorded against api.devcloud.linode.com).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -14,308 +14,31 @@ interactions:
url: https://api.linode.com/v4beta/regions?page=1
@@ -14,308 +14,31 @@ interactions:
url: https://api.linode.com/v4beta/regions?page=1
@@ -14,169 +14,275 @@ interactions:
url: https://api.linode.com/v4beta/images?page=1
@@ -14,183 +14,275 @@ interactions:
url: https://api.linode.com/v4beta/images?page=1
- application/json
User-Agent:
- linodego/dev https://github.com/linode/linodego
url: https://api.linode.com/v4beta/regions?page=1
@lgarber-akamai
Copy link
Copy Markdown
Contributor

@psnoch-akamai Any chance you could regenerate the fixtures against production to get the CI passing? Alternatively you can manually replace all DevCloud API URLs in the fixture files with api.linode.com

@psnoch-akamai psnoch-akamai force-pushed the TPT-4435-upgrade-linode-alpine3.19-to-linode-alpine3.23-for-linodego-tests branch from a1b4195 to 6c87bc0 Compare May 14, 2026 12:30
Copy link
Copy Markdown
Contributor

@lgarber-akamai lgarber-akamai left a comment

Choose a reason for hiding this comment

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

Thanks for handling this!

@psnoch-akamai psnoch-akamai force-pushed the TPT-4435-upgrade-linode-alpine3.19-to-linode-alpine3.23-for-linodego-tests branch from 6c87bc0 to f4f6a94 Compare May 14, 2026 14:43
@psnoch-akamai psnoch-akamai force-pushed the TPT-4435-upgrade-linode-alpine3.19-to-linode-alpine3.23-for-linodego-tests branch from f4f6a94 to c6dad9c Compare May 14, 2026 15:04
@psnoch-akamai psnoch-akamai merged commit 56329bb into linode:main May 14, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants