-
Notifications
You must be signed in to change notification settings - Fork 4
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
[Cicognara] Removes inventory files #5655
Conversation
Co-authored-by: Vickie Karasic <[email protected]> Co-authored-by: Angel Ruiz <[email protected]>
@@ -12,12 +12,6 @@ named_sites: | |||
repo_root: "/var/local/daviesproject" | |||
git_repo: 'https://github.com/pulibrary/davies_project.git' | |||
version: 'main' | |||
- server_name: "cicognara.org" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
won't this bring down cicognara?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indeed, the entries in libstatic need to stay
@@ -2,12 +2,6 @@ | |||
# defaults file for roles/libstatic | |||
running_on_server: true | |||
named_sites: | |||
- server_name: 'cicognara-staging.lib' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we want cicognara staging to continue to run.
@@ -10,12 +10,6 @@ named_sites: | |||
repo_root: "/var/local/milberg" | |||
git_repo: 'https://github.com/pulibrary/milberg-wp.git' | |||
version: 'main' | |||
- server_name: 'cicognara-staging.lib' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we want this to stay.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
anything in libstatic should stay
@@ -132,7 +132,3 @@ | |||
# lae redirects | |||
# | |||
rewrite ^/collections/pudl0025(.*)$ https://lae.princeton.edu$1 redirect; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will remove cicognara
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we want cicognara to remain as a static site for both staging and prod
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cicognara.org is transferring from a standalone site to an entry in our static sites. We need to get rid of the old standalone infrastructure and automation. We need to keep the automation that relates to its entry in the static sites list, though.
@@ -1,15 +1,11 @@ | |||
--- | |||
# by default this playbook runs in the staging environment | |||
# to run in production, pass '-e runtime_env=production' | |||
- name: build the cicognara site |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This playbook can be deleted entirely. We do not want to build a standalone cicognara site any more.
@@ -10,12 +10,6 @@ named_sites: | |||
repo_root: "/var/local/milberg" | |||
git_repo: 'https://github.com/pulibrary/milberg-wp.git' | |||
version: 'main' | |||
- server_name: 'cicognara-staging.lib' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
anything in libstatic should stay
@@ -11,15 +11,6 @@ upstream static-staging { | |||
zone=static-stagingclient_sessions:1m; | |||
} | |||
|
|||
server { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these need to stay also
PR #5714 closes this PR |
retake #5714 replaces this PR |
Closes #5586
Co-authored-by: Vickie Karasic [email protected]
Co-authored-by: Angel Ruiz [email protected]