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

Add a step for Minikube deployment instructions #157

Merged
merged 1 commit into from
Jan 14, 2020

Conversation

adinin
Copy link
Contributor

@adinin adinin commented Jan 14, 2020

Issue

In Minikube deployment, the Contour NodePort does not become readily available outside the cluster.
Hence, a statement

For local deployments, this endpoint is typically http://localhost:30081/console.

is misleading to Minikube users.

Solution

Adding an extra instruction for Minikube users clarifies how to access the console.

Copy link
Contributor

@katrogan katrogan left a comment

Choose a reason for hiding this comment

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

Thanks for adding!

@katrogan katrogan merged commit 2f3c1b5 into flyteorg:master Jan 14, 2020
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Dec 6, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Dec 6, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Dec 20, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Jul 24, 2023
* Add a new requestID to Log fields

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* Also set the requestID on grpc outgoing requests

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* set key and value

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* lint :(

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* Use more commonly used string for request id

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* using request-id instead per PR Comment discussion

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* reverting to x- prefix to keep OOB behavior sane

Signed-off-by: Haytham Abuelfutuh <[email protected]>

---------

Signed-off-by: Haytham Abuelfutuh <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Aug 9, 2023
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Aug 21, 2023
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Aug 21, 2023
* Add a new requestID to Log fields

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* Also set the requestID on grpc outgoing requests

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* set key and value

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* lint :(

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* Use more commonly used string for request id

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* using request-id instead per PR Comment discussion

Signed-off-by: Haytham Abuelfutuh <[email protected]>

* reverting to x- prefix to keep OOB behavior sane

Signed-off-by: Haytham Abuelfutuh <[email protected]>

---------

Signed-off-by: Haytham Abuelfutuh <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Apr 30, 2024
austin362667 pushed a commit to austin362667/flyte that referenced this pull request May 7, 2024
robert-ulbrich-mercedes-benz pushed a commit to robert-ulbrich-mercedes-benz/flyte that referenced this pull request Jul 2, 2024
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.

2 participants