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
Add stack endpoints #1866
Add stack endpoints #1866
Conversation
Signed-off-by: Javier López Barba <javier@okteto.com>
Signed-off-by: Javier López Barba <javier@okteto.com>
Codecov Report
@@ Coverage Diff @@
## master #1866 +/- ##
==========================================
- Coverage 33.62% 33.46% -0.17%
==========================================
Files 122 124 +2
Lines 13758 13825 +67
==========================================
Hits 4626 4626
- Misses 8632 8699 +67
Partials 500 500
Continue to review full report at Codecov.
|
@jLopezbarb could you share a screenshot of how does it look like? |
Signed-off-by: Javier López Barba <javier@okteto.com>
Why don't you show the endpoint after the successful message? And it could be an "info" line type. |
Thank you for your contribution. unfortunately, one or more of your commits are missing the required "Signed-off-by:" statement. Signing off is part of the Developer Certificate of Origin (DCO) which is used by this project. Read the DCO and project contributing guide carefully, and amend your commits using the git CLI. Note that this does not require any cryptography, keys or special steps to be taken. 💡 Shall we fix this?This will only take a few moments. First, clone your fork and checkout this branch using the git CLI. Next, set up your real name and email address:
Finally, run one of these commands to add the "Signed-off-by" line to your commits. If you only have one commit so far then run: Check that the message has been added properly by running "git log". |
Signed-off-by: Javier López Barba <javier@okteto.com>
I feel like that last part is too verbose. We show the stack name in both messages. Maybe the second one could just say "Endpoints available:" |
Signed-off-by: Javier López Barba <javier@okteto.com>
@jLopezbarb Agree with Ramiro. Update the screenshot when ready. |
@rberrelleza isn't it more correct to say "Available endpoints"? |
Not sure tbh. Maybe we should just use |
They are both correct but I prefer It's also listed on wikipedia as an example of postpositive adjective use :) It's one of those things I've never thought about but once I do I have no clue how English works, https://en.wikipedia.org/wiki/Postpositive_adjective, |
@jmacelroy English can be a bit of a nightmare when it comes to exceptions or special cases... In that case, I like the |
) | ||
|
||
func ListEndpoints(ctx context.Context, stack *model.Stack, output string) error { | ||
oktetoClient, err := okteto.NewOktetoClient() |
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.
@jLopezbarb this is wrong. We could list the ingresses instead and get the URL from there. That wway endpoints are available in vanilla kubernetes clusters too
Signed-off-by: Javier López Barba javier@okteto.com
Proposed changes
okteto stack deploy --wait