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

Adding unity_proxy airfow proxy config #170

Merged
merged 7 commits into from
Jul 29, 2024

Conversation

jpl-btlunsfo
Copy link
Collaborator

@jpl-btlunsfo jpl-btlunsfo commented Jul 14, 2024

Purpose

  • Clear, easy-to-understand sentences outlining the purpose of the PR

Proposed Changes

  • ADD unity-proxy httpd configuration ssm parameter for Airflow endpoint
  • ADD matching lambda invocation for the above proxy container restart

Issues

Testing

Copy link
Collaborator

@galenatjpl galenatjpl left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@jpl-btlunsfo jpl-btlunsfo force-pushed the 125-airfow-httpd-proxy-config branch from 1a05953 to 91e5584 Compare July 22, 2024 18:06
@jpl-btlunsfo
Copy link
Collaborator Author

Rebased, and added proxy configuration for OGC endpoint

@jpl-btlunsfo jpl-btlunsfo force-pushed the 125-airfow-httpd-proxy-config branch from 49d8c95 to 321ea98 Compare July 29, 2024 16:33
@jpl-btlunsfo
Copy link
Collaborator Author

@LucaCinquini should be safe to merge now!

A note, the configuration is scoped to the project and venue, so you'll need to make sure you spin up a MC instance for the same project/venue before you initialize an airflow instance for demo/testing.

@LucaCinquini LucaCinquini merged commit 870310b into develop Jul 29, 2024
2 checks passed
@LucaCinquini LucaCinquini deleted the 125-airfow-httpd-proxy-config branch July 29, 2024 18:42
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.

3 participants