#3082 closed task (fixed)
Contributions to ldap-web
| Reported by: | cvvergara | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Sysadmin Contract 2024-II |
| Component: | SysAdmin/LDAP | Keywords: | |
| Cc: |
Description (last modified by )
We have a second PR on the last 30 days, to ldap-web repositories, and they are difficult to test. (with and without ansible)
Needed: changes on the way ansible & how to do contributions work together so that a contribution gets tested without the need of ansible, and gets deployed into production with ansible.
Will be using web-cgi-bin/pulls/25 as test base
Change History (6)
comment:1 by , 11 months ago
| Description: | modified (diff) |
|---|
comment:2 by , 11 months ago
Created a test script, but it needs credentials and I am getting a 500 error here https://git.osgeo.org/gitea/user/settings/keys
comment:3 by , 5 months ago
| Component: | SysAdmin → SysAdmin/LDAP |
|---|
I think Ansible should still be involved in testing as you would:
- Deploy to staging (via Ansible)
- Test staging
- When happy: deploy to production (via Ansible)
Vicky: the internal server error on SSH keys Gitea page is now fixed. In any case you add the SSH keys via LDAP, the Gitea page just directs you there ( https://id.osgeo.org/ldap/edit )
comment:4 by , 2 months ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
The PR from Jorge was merged. Looks like no problems further down the road.
comment:6 by , 2 months ago
I see the changes in staging, but they are not correctly styled as the two CSS files are still missing even after https://git.osgeo.org/gitea/sac/ansible-deployment/pulls/57
:(
Full URL to test base: https://git.osgeo.org/gitea/sac/web-cgi-bin/pulls/25