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

Themes #194

Merged
merged 25 commits into from
Nov 23, 2023
Merged

Themes #194

merged 25 commits into from
Nov 23, 2023

Conversation

mira-miracoli
Copy link
Collaborator

I included the ansible-galaxy-subdomains role in this one and updated it to the new galaxy-integrated per_host functionality. See

@mira-miracoli
Copy link
Collaborator Author

I think the failing tests might be caused by broken dependencies in dev

Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

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

I am very excited for this!

defaults/main.yml Outdated Show resolved Hide resolved
tasks/setup_static_dirs.yml Outdated Show resolved Hide resolved
@hexylena
Copy link
Member

hexylena commented Sep 4, 2023

@natefoo can you give this a look?

Copy link
Member

@natefoo natefoo left a comment

Choose a reason for hiding this comment

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

Awesome stuff, thank you!

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
defaults/main.yml Show resolved Hide resolved
tasks/copy_static_files.yml Outdated Show resolved Hide resolved
tasks/setup_static_dirs.yml Outdated Show resolved Hide resolved
tasks/setup_static_dirs.yml Outdated Show resolved Hide resolved
Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

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

Ok, I think i've found a workaround for setting the variables, this is doable!

defaults/main.yml Show resolved Hide resolved
defaults/main.yml Outdated Show resolved Hide resolved
@natefoo
Copy link
Member

natefoo commented Nov 21, 2023

The archive/unarchive is going to result in a lot of always-run tasks, I think? But I also recall you mentioned some reason that using the synchronize module won't work?

@mira-miracoli
Copy link
Collaborator Author

The archive/unarchive is going to result in a lot of always-run tasks, I think? But I also recall you mentioned some reason that using the synchronize module won't work?

I don't know anymore why it did not work in the past. Now it works :D

@mira-miracoli
Copy link
Collaborator Author

Are you ok with the latest changes?

Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

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

This looks fantastic and much more idemopotent! A couple nitpicky comments but this feels basically ready to ship

defaults/main.yml Show resolved Hide resolved
tasks/setup_static_dirs.yml Outdated Show resolved Hide resolved
templates/welcome.html.j2 Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
defaults/main.yml Show resolved Hide resolved
tasks/main.yml Outdated
Comment on lines 163 to 175

- name: Create welcome.html directory for basedomain
ansible.builtin.file:
state: directory
mode: '0755'
owner: "{{ __galaxy_privsep_user_name }}"
group: "{{ __galaxy_privsep_user_group }}"
path: "{{ galaxy_themes_static_path }}/static/welcome.html"
when: galaxy_manage_subdomain_static

- name: Template welcome.html for basedomain
ansible.builtin.template:
src: welcome.html.j2
Copy link
Member

Choose a reason for hiding this comment

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

I think all of the when: galaxy_manage_subdomain_static could be moved into a separate file to match the rest of main.yml that does conditional includes.

mira-miracoli and others added 5 commits November 23, 2023 10:00
Co-authored-by: Helena <hexylena@galaxians.org>
Co-authored-by: Helena <hexylena@galaxians.org>
Co-authored-by: Helena <hexylena@galaxians.org>
Co-authored-by: Helena <hexylena@galaxians.org>
@hexylena hexylena merged commit 20ac2d5 into galaxyproject:main Nov 23, 2023
4 of 15 checks passed
@hexylena
Copy link
Member

🎉

@kysrpex
Copy link

kysrpex commented Nov 23, 2023

🎉

This was a long one, congrats to you all 🚀!

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.

None yet

4 participants