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

Update bdii-slapd-start for IPv6 support #51

Merged
merged 2 commits into from
May 16, 2024

Conversation

ellert
Copy link
Contributor

@ellert ellert commented May 16, 2024

Summary

I would like to propose an alternative solution to #50. I find this solution less confusing since it does not use bash arrays.

Related issue : #50

@ellert ellert requested a review from a team as a code owner May 16, 2024 04:27
@brucellino brucellino self-assigned this May 16, 2024
@brucellino
Copy link
Member

Hi @ellert ! Thanks very much.

I will take a look at this too and let you know. Since we can either merge #48 or #51 but not both, I want to have a closer look and add a functional test.

I'll get back in a few days.

@brucellino
Copy link
Member

So far I have tested the change with the artifact built in this run:

  • the configuration file is ok
  • slapd listens on IPv6 on lo the public interface --
[root@almalinux-s-1vcpu-1gb-ams3-01 ~]# ldapsearch -x -H "ldap://[2a03:b0c0:2:d0::1712:3001]:2170" -b 'Mds-Vo-Name=$SITE_NAME,o=Grid'  '(ObjectClass=GlueSite)'
# extended LDIF
#
# LDAPv3
# base <Mds-Vo-Name=$SITE_NAME,o=Grid> with scope subtree
# filter: (ObjectClass=GlueSite)
# requesting: ALL
#

# search result
search: 2
result: 32 No such object
matchedDN: o=grid

# numResponses: 1

So I think we can merge.

@brucellino brucellino merged commit 5287f46 into EGI-Federation:main May 16, 2024
8 checks passed
@ellert ellert deleted the slapd-ipv6 branch May 16, 2024 20:17
@gwarf gwarf mentioned this pull request Jun 14, 2024
ellert pushed a commit that referenced this pull request Jun 15, 2024
# [6.0.2] - 2024-06-14

# Changed

- Fix for IPv6 support. (#51) (Mattias Ellert)
- Replace obsolete -h and -p paremeters in ldap CLI tools. (#49)
(Baptiste Grenier)
- Fix deprecation warning due to log.warn. (#58) (Daniela Bauer)
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.

2 participants