Replaced deprecated logic in SPCS worker

[bjorhn]

This PR updates the Snowpark Container Services worker to handle recent changes in the API of the Snowflake Python SDK:

• Removed usage of deprecated method get_service_status()
• Updated status checks to account for changes in status types returned by the API
• Bumps Snowflake dependency to >= 1.5.0 (from >= 1.2.0)

There are no new unit tests since there is no new functionality.

This code is running in production at my workplace with no issues.

Closes #18104

Checklist

• This pull request references any related issue by including "closes <link to issue>"
  • If no issue exists and your change is not a small fix, please create an issue first.
• If this pull request adds new functionality, it includes unit tests that cover the changes
• If this pull request removes docs files, it includes redirect settings in mint.json.
• If this pull request adds functions or classes, it includes helpful docstrings.

[zzstoatzz]

hi @bjorhn it looks like we need to update a test here, otherwise this LGTM

[zzstoatzz]

hi @bjorhn it looks like we need to update a test here, otherwise this LGTM

here's the failure im looking at

[bjorhn]

@zzstoatzz I'm not sure how any of my changes relate to that test, am I missing something? Is it possible that my snowflake dependency bump is forcing a higher version of snowflake-connector, which, in turn, causes the test to fail?

[zzstoatzz]

@zzstoatzz I'm not sure how any of my changes relate to that test, am I missing something? Is it possible that my snowflake dependency bump is forcing a higher version of snowflake-connector, which, in turn, causes the test to fail?

@bjorhn yea i would guess the new versions just handle that PEM loading error a bit different so i suspect we'd just need to update the expected error message we're matching on now

if you get stuck on that I can take a look later

[bjorhn]

@zzstoatzz I had a quick look at this but at the moment I don't know when I can sit down and get this fixed. PEM loading isn't really my area of expertise. If you are able to take a look at this, please do, otherwise I will probably return to it at a later date.

[zzstoatzz]

ok @bjorhn I will take a look at this later this week

[zzstoatzz]

the diff is massive here because of: astral-sh/uv#13176

its all just changing upload_time to upload-time - the latter is new and correct

[zzstoatzz]

thank you @bjorhn - apologies for the delay here