Skip to content

feat(vefaas): get application endpoint#35

Merged
cuericlee merged 2 commits intovolcengine:mainfrom
zakahan:main
Aug 7, 2025
Merged

feat(vefaas): get application endpoint#35
cuericlee merged 2 commits intovolcengine:mainfrom
zakahan:main

Conversation

@zakahan
Copy link
Copy Markdown
Collaborator

@zakahan zakahan commented Aug 7, 2025

Testing plan

  • You can use this section of code to test this PR
from veadk.cloud.cloud_app import CloudApp
cloud = CloudApp(
    vefaas_application_id="",           # Any one of them can be empty, but must set one of them.
    vefaas_application_name="",
)
url = cloud._get_vefaas_endpoint(
    volcengine_ak="your-ak",
    volcengine_sk="your-sk"
)
print(url)
  • add typer on requirements.txt, for veadk web of vefaas ( cli need this)

@zakahan zakahan requested a review from yaozheng-fang August 7, 2025 04:23
cuericlee
cuericlee reviewed Aug 7, 2025
View reviewed changes
veadk/cli/services/vefaas/vefaas.py Outdated
)
return response["Result"]["Items"]

def _get_application_details(self, app_id: str = None, app_name: str = None):
Copy link
Copy Markdown
Collaborator

@cuericlee cuericlee Aug 7, 2025

Choose a reason for hiding this comment

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

public method?

Copy link
Copy Markdown
Collaborator Author

@zakahan zakahan Aug 7, 2025

Choose a reason for hiding this comment

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

fixed

@zakahan zakahan requested a review from cuericlee August 7, 2025 05:04
cuericlee
cuericlee approved these changes Aug 7, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@cuericlee cuericlee left a comment

Choose a reason for hiding this comment

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

/lgtm

cuericlee
cuericlee approved these changes Aug 7, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@cuericlee cuericlee left a comment

Choose a reason for hiding this comment

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

/lgtm

@cuericlee cuericlee merged commit b271fb7 into volcengine:main Aug 7, 2025
5 checks passed
yaozheng-fang pushed a commit that referenced this pull request Aug 15, 2025
@zakahan
feat(vefaas): get application endpoint (#35)
40a951d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cuericlee cuericlee cuericlee approved these changes

@yaozheng-fang yaozheng-fang Awaiting requested review from yaozheng-fang

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@zakahan @cuericlee