Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

fix(sg/eph): handle nil regex#62764

Merged
burmudar merged 1 commit intomainfrom
wb/sg/eph-panic
May 17, 2024
Merged

fix(sg/eph): handle nil regex#62764
burmudar merged 1 commit intomainfrom
wb/sg/eph-panic

Conversation

@burmudar
Copy link
Contributor

if you didn't provide a regex this during filtering list-versions would break. This slipped through while doing last minute refactoring.

go run ./dev/sg cloud eph list-versions
☁️ Retrieved 543 docker images from registry "cloud-ephemeral"
Tag                                                Upload time          Image count
b6a5c7dd76f292ce90a6d3c4c029f072ad41848b_272681... 2024-03-15 11:19:33  48
wb-sg-cloud-eph-delete_271241_2024-04-29_5.3-76... 2024-03-15 11:19:33  48
2c328e9e7e08_265572                                2024-03-15 11:19:17  45
23fafbf0fa90eff28fe5ba8b3cdcd3bd09a313d3_269589... 2024-03-15 11:19:33  46
e90a171118c9_2024-03-15                            2024-03-15 11:19:17  45
6d88b156f78664c3a73b45693473ae7de58b25e4_270425... 2024-03-15 11:19:33  47

Test plan

Tested locally

@burmudar burmudar requested review from a team and filiphaftek May 17, 2024 13:54
@burmudar burmudar self-assigned this May 17, 2024
@cla-bot cla-bot bot added the cla-signed label May 17, 2024
@burmudar burmudar merged commit 0c30242 into main May 17, 2024
@burmudar burmudar deleted the wb/sg/eph-panic branch May 17, 2024 14:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants