-
Notifications
You must be signed in to change notification settings - Fork 2.2k
feature(main): add pyament deploy #2035
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
Conversation
Signed-off-by: cuisongliu <[email protected]>
Codecov ReportBase: 67.98% // Head: 67.98% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #2035 +/- ##
=======================================
Coverage 67.98% 67.98%
=======================================
Files 6 6
Lines 506 506
=======================================
Hits 344 344
Misses 135 135
Partials 27 27 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
||
### How to run | ||
|
||
```shell | ||
sealos run docker.io/labring/kustomize:v4.5.6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems we don't need the callback URL anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed it to NotifyCallbackURL
. It is actually a callback interface for payment to inform us of the status. I think it's more appropriate to keep it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use Notify too, controller will check the payment status every 0.3s, so needn't any notify
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use Notify too, controller will check the payment status every 0.3s, so needn't notify
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use Notify too, controller will check the payment status every 0.3s, so needn't any notify
OK,Let me fix this problem.
d837dbf
to
fd72ec8
Compare
Signed-off-by: cuisongliu <[email protected]>
fd72ec8
to
ebbba45
Compare
Signed-off-by: cuisongliu [email protected]