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

SCP-3176 support flink artifact on azure #2968

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

SCP-3176 support flink artifact on azure #2968

wants to merge 1 commit into from

Conversation

cjuyin
Copy link
Contributor

@cjuyin cjuyin commented Dec 9, 2024

Release Notes

Breaking Changes

  • PLACEHOLDER

New Features

  • PLACEHOLDER

Bug Fixes

  • PLACEHOLDER

Checklist

  • Leave this box unchecked if features are not yet available in production

What

We need to support flink on Azure. Modify the uploadFile to support PUT method.
Presigned url API failed initially and works after we support PUT method.

dist/confluent_darwin_arm64_v8.0/confluent  flink artifact create juyin-azure-1 --artifact-file ~/Downloads/3KB.jar --cloud azure --region centralus --environment env-stgc3ddo3m
Error: <?xml version="1.0" encoding="utf-8"?>
<Error><Code>UnsupportedHttpVerb</Code><Message>The resource doesn&apos;t support specified Http Verb.
RequestId:16893274-101e-0033-2b5f-4ab7d4000000
Time:2024-12-09T17:24:16.5775453Z</Message></Error>

dist/confluent_darwin_arm64_v8.0/confluent  flink artifact create juyin-azure-1 --artifact-file ~/Downloads/3KB.jar --cloud azure --region centralus --environment env-stgc3ddo3m
+--------------------+----------------+
| ID                 | cfa-stgc41gdwv |
| Name               | juyin-azure-1  |
| Version            | ver-stgce056yp |
| Cloud              | azure          |
| Region             | centralus      |
| Environment        | env-stgc3ddo3m |
| Content Format     | JAR            |
| Description        |                |
| Documentation Link |                |
+--------------------+----------------+


List, Get, Delete work as expected.

juyinchung@MY26316PPV cli % dist/confluent_darwin_arm64_v8.0/confluent  flink artifact describe cfa-stgc41gdwv --cloud azure --region centralus --environment env-stgc3ddo3m
+--------------------+----------------+
| ID                 | cfa-stgc41gdwv |
| Name               | juyin-azure-1  |
| Version            | ver-stgce056yp |
| Cloud              | AZURE          |
| Region             | centralus      |
| Environment        | env-stgc3ddo3m |
| Content Format     | JAR            |
| Description        |                |
| Documentation Link |                |
+--------------------+----------------+
juyinchung@MY26316PPV cli % dist/confluent_darwin_arm64_v8.0/confluent  flink artifact list --cloud azure --region centralus --environment env-stgc3ddo3m
        ID       |         Name          | Cloud |  Region   |  Environment
-----------------+-----------------------+-------+-----------+-----------------
  cfa-stgc41gdwv | juyin-azure-1         | AZURE | centralus | env-stgc3ddo3m
  cfa-stgc4jqv02 | juyin-internal-shim-1 | AZURE | centralus | env-stgc3ddo3m
  cfa-stgce85vqr | juyin-internal-shim   | AZURE | centralus | env-stgc3ddo3m
  cfa-stgcl6mw7j | azure-artifact        | AZURE | centralus | env-stgc3ddo3m
juyinchung@MY26316PPV cli % dist/confluent_darwin_arm64_v8.0/confluent  flink artifact delete cfa-stgc41gdwv cfa-stgc4jqv02 cfa-stgce85vqr  --cloud azure --region centralus --environment env-stgc3ddo3m
Are you sure you want to delete Flink artifacts "cfa-stgc41gdwv", "cfa-stgc4jqv02", and "cfa-stgce85vqr"?
This action is irreversible and is going to break all Flink statements using this Artifact!
Do you still want to proceed? (y/n): y
Deleted Flink artifacts "cfa-stgc41gdwv", "cfa-stgc4jqv02", and "cfa-stgce85vqr".

References

Test & Review

@cjuyin cjuyin requested a review from a team as a code owner December 9, 2024 17:39
@confluent-cla-assistant
Copy link

🎉 All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

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.

1 participant