Skip to content
This repository has been archived by the owner on May 31, 2024. It is now read-only.

#3458 Include default MinIO credentials #410

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Conversation

GRomR1
Copy link
Contributor

@GRomR1 GRomR1 commented Jun 29, 2023

#3458 Include default MinIO credentials in sandbox console readout

TL;DR

Include default MinIO credentials in sandbox console readout

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

When following the steps to setup Flyte on a demo sandbox, the console reports that 📂 The Minio API is hosted on localhost:30002. Use http://localhost:30080/minio/login for Minio console. However, in order to view the obect store, you need to use the default credentials for MinIO, which are:

username: minio
password: miniostorage

Tracking Issue

flyteorg/flyte#3458

#3458 Include default MinIO credentials in sandbox console readout

Signed-off-by: Ruslan Gainanov <[email protected]>
@oliverhu
Copy link

oliverhu commented Jul 3, 2023

+1 to this PR...I had to decode the secrets to get these credentials. Maybe put these in Constants in the same file than hard coding in the message?

@pingsutw pingsutw merged commit 8428367 into flyteorg:master Nov 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants