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

v1.7.1 Add missing PERL dependency on el8. Drop support of RHEL7. #21

Merged
merged 9 commits into from
Nov 15, 2024

Conversation

gwarf
Copy link
Member

@gwarf gwarf commented Nov 15, 2024

  • Add missing PERL dependency on el8.
  • Drop support for RHEL7.
  • Release version 1.7.1.

Fixes #20.

@gwarf gwarf requested review from a team as code owners November 15, 2024 06:21
@gwarf gwarf changed the title Add missing perl dependency on ekl8 Add missing perl dependency on el8 Nov 15, 2024
@gwarf gwarf changed the title Add missing perl dependency on el8 Add missing PERL dependency on el8. Drop support of RHEL7. Nov 15, 2024
@gwarf gwarf changed the title Add missing PERL dependency on el8. Drop support of RHEL7. v1.7.1 Add missing PERL dependency on el8. Drop support of RHEL7. Nov 15, 2024
Copy link

@enolfc enolfc left a comment

Choose a reason for hiding this comment

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

LGTM, maybe after install we could do a simple call of the command just to check that it runs?

@gwarf
Copy link
Member Author

gwarf commented Nov 15, 2024

LGTM, maybe after install we could do a simple call of the command just to check that it runs?

Yes was thinking of this, tried earlier in a docker and calling the script should at least display the help after having loaded the deps:

[root@99489f2ad40a source]# /usr/libexec/glite-info-provider-ldap
Usage: /usr/libexec/glite-info-provider-ldap -c <config file> [-m <mds-vo-name>] [-v validity] -[d dynamic] [-t timeout] [-s size] [-g site/region]

This information provider takes a list of LDAP urls from the configuration
file and queries the LDAP sources. It then merges the results and modifies
the dn to append the mds-vo-name as specified on the command line.
The configuration file has lines typically with the following format:

entity  ldap://host.domain:2170/mds-vo-name=something,o=grid

Comments start with "#" and are ignored.

Options:

    -c  The configuration file listing all the LDAP URLs.

    -m  The mds-vo-name which should be used.

    -h  Displays this helpful message.

    -d  This option will change the search filter to only
        retrieve the dynamic information. (currently not supported)

    -g  This option will modify the search endpoint to obtain glue 2.0 
        information and dynamically modify the dn if value is not "none".

    -t  The timeout of the ldapsearch in seconds.

    -v  The validity of the cache files in seconds.

    -s  Maximum file size in megabytes for a single source.

But I won't be able to implement this today, so in case anyone wants to have a look... :)

@gwarf
Copy link
Member Author

gwarf commented Nov 15, 2024

So I've added a step to the build workflow to confirm the scripts load and displays the help.

@gwarf gwarf merged commit bd87195 into EGI-Federation:main Nov 15, 2024
6 checks passed
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.

glite-info-provider-ldap-1.6.1-1.el8 misses dependencies
2 participants