Skip to content

add webserver quick link for airflow up#43

Merged
schnie merged 2 commits intomasterfrom
feature/37-output-webserver-link
Mar 6, 2018
Merged

add webserver quick link for airflow up#43
schnie merged 2 commits intomasterfrom
feature/37-output-webserver-link

Conversation

@andscoop
Copy link
Contributor

@andscoop andscoop commented Mar 5, 2018

Quick change to make address of the airflow webserver easier to acccess

closes #37

@andscoop andscoop requested a review from schnie March 5, 2018 20:16
@schnie
Copy link
Member

schnie commented Mar 5, 2018

@andscoop want to add another line saying where the postgres database is running too? Nice to be able connect to it to tinker as a user.

@andscoop
Copy link
Contributor Author

andscoop commented Mar 6, 2018

@schnie I have updated with requested changes.

@andscoop andscoop force-pushed the feature/37-output-webserver-link branch from 5c6a475 to e6242ae Compare March 6, 2018 15:41
}

fmt.Println("Airflow Webserver: http://localhost:8080/admin/")
fmt.Println("Postgres Database: 0.0.0.0:5432/postgres")
Copy link
Member

Choose a reason for hiding this comment

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

@andscoop can we change 0.0.0.0 to localhost?

@andscoop andscoop force-pushed the feature/37-output-webserver-link branch from e6242ae to 4aab7aa Compare March 6, 2018 16:22
@andscoop
Copy link
Contributor Author

andscoop commented Mar 6, 2018

@schnie updated

Copy link
Member

@schnie schnie left a comment

Choose a reason for hiding this comment

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

👌

@schnie schnie merged commit e9f9466 into master Mar 6, 2018
@andscoop andscoop deleted the feature/37-output-webserver-link branch April 18, 2018 01:31
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.

Output Links after a Successful astro airflow up

2 participants