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

remove deploy_request && add deployment_input_properties #2422

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

baixinsui
Copy link
Contributor

@baixinsui baixinsui commented Feb 18, 2025

Fixed #2346
Fixed #2331
Related to eclipse-xpanse/xpanse-ui#1529

Deploy service successfully:
image
Details of service:
image
Details of deploy order:
image

Modify service successfully:
image
Details of modify order:
image

Port service successfully:
image
Details of port order:
image

Recreate service successfully:
image
Details of recreate order:
image
Details of lockConfig order:
image
Details of serviceRestart order:
image

@Alice1319
Copy link
Contributor

LGTM

1 similar comment
@yuanxiaojun987
Copy link

LGTM

@baixinsui baixinsui force-pushed the fix/remove_deploy_request branch 9 times, most recently from f2cbd4f to 620eda5 Compare February 21, 2025 08:49
@ruwenqiang123
Copy link

LGTM

@baixinsui baixinsui force-pushed the fix/remove_deploy_request branch from 620eda5 to 29c28e7 Compare February 24, 2025 02:31
@baixinsui baixinsui force-pushed the fix/remove_deploy_request branch from 29c28e7 to f2ec435 Compare February 24, 2025 03:19
@baixinsui baixinsui closed this Feb 24, 2025
@baixinsui baixinsui reopened this Feb 24, 2025
@baixinsui baixinsui requested a review from swaroopar February 24, 2025 06:15
Copy link
Contributor

@swaroopar swaroopar left a comment

Choose a reason for hiding this comment

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

LGTM

@swaroopar swaroopar merged commit f8f9316 into eclipse-xpanse:main Feb 24, 2025
4 of 5 checks passed
@qigaoneng
Copy link

LGTM

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.

remove redundant columns from SERVICE_DEPLOYMENT wrong request body stored for destroy order
6 participants