-
Notifications
You must be signed in to change notification settings - Fork 0
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
Get Airflow working inside Navbar #22
Comments
@LucaCinquini what info do you need from us to configure Airflow to be happy with CORS requests from the navbar UI to Airflow? Can we read the location from SSM? |
Yes, the location is in SSM and @drewm-jpl can tell you which is the parameter to use. But I think the UI/UX team should try to include it in the NavBar and see what problems arise. As I mentioned earlier, I am not sure it is such a great idea to imprison Airflow within another site... |
To find the Airflow UI URL in a given venue, you can find it in an SSM parameter under this naming convention |
@drewm-jpl My understanding of the param names is that it should follow something like |
@anilnatha Whoops, yes that's correct the venue is parameterized in the SSM parameter name as well.
Currently, project and venue are combined so the SBG dev SSM parameter would appear like this: There doesn't appear to be guidance on the gitbook docs on how to handle the project/venue combo but that's what we have currently. |
@drewm-jpl Thanks for the clarification and addressing my concern. @galenatjpl Could you please point us to the documentation on how to handle the SSM param naming convention with regards to project and venue. IIRC, it was something like |
@anilnatha I think we are now settling on |
@anilnatha please work with @drewm-jpl to update this (e.g. use the health json landingPage URL?) or otherwise get a URL to try out to test this... (assuming the CORS stuff will work) |
@anilnatha and @rtapella ping for status. Is this still blocked? |
Once Airflow is deployed with the new URL structure, we can test this out—that will/should avoid CORS issues. Initially we'll just use the URL from the Health Status info for each app. |
@LucaCinquini @drewm-jpl we will need the Airflow health-endpoint URLs to have the MCP URL |
@LucaCinquini Please add sub-issues to this Epic as needed to address the tasks needed to get Airflow accessible from the navbar (aka portal) |
So do you simply need a link to the Airflow URL? Isn't that accessible already through the Health Check endpoint? |
Are you trying to display Airflow in an embedded window? Otherwise, the URL works just fine. |
Yup, that's exactly what we're trying to do. |
No description provided.
The text was updated successfully, but these errors were encountered: