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

ncm-freeipa: support freeipa 4.7 #1403

Merged
merged 5 commits into from
Jul 28, 2023
Merged

ncm-freeipa: support freeipa 4.7 #1403

merged 5 commits into from
Jul 28, 2023

Conversation

stdweird
Copy link
Member

@stdweird stdweird commented Nov 3, 2019

Support some minor differences wrt verison 4.7

@jrha jrha added this to the 20.3 milestone Mar 19, 2020
jrha
jrha previously approved these changes Mar 19, 2020
jrha
jrha previously approved these changes Oct 8, 2020
Copy link
Member

@jrha jrha left a comment

Choose a reason for hiding this comment

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

LGTM, does seem to be failing tests though.

@stdweird
Copy link
Member Author

stdweird commented Oct 9, 2020

@jrha any idea how to retrigger the tests? i'll see if i can do this with close/open PR

@stdweird stdweird closed this Oct 9, 2020
@stdweird stdweird reopened this Oct 9, 2020
@jrha
Copy link
Member

jrha commented Oct 9, 2020

Looks like ${rpm.release} isn't being expanded when the tests are run:

#   Failed test 'install freeipa component and CLI dependencies in post_reboot'
#   at src/test/perl/aii-basic.t line 61.
#                   '[VERB] AII role aii using predefined principal quattor-aii keytab /etc/quattor-aii.keytab
# [DEBUG] Creating Kerberos instance
# [DEBUG] update_env KRB5_CLIENT_KTNAME = /etc/quattor-aii.keytab.
# [DEBUG] update_env KRB5_KTNAME = /etc/quattor-aii.keytab.
# [DEBUG] Creating FreeIPA::Client instance
# [DEBUG] Login using kerberos
# [DEBUG] Successful login
# [DEBUG] set_api_version using API version v2.230
# [DEBUG] set api_version to 2.230
# [DEBUG] Successful JSON POST
# [DEBUG] api_host_add: successfully add-ed host myhost.example.com
# [DEBUG] Successful JSON POST
# [DEBUG] api_host_mod: successfully mod-ed host myhost.example.com
# # freeipa KS post_reboot
# echo 'begin freeipa post_reboot'
# 
# yum -y install ncm-freeipa-19.12.1-${rpm.release} nss-pam-ldapd ipa-client nss-tools openssl pam_krb5 -c /tmp/aii/yum/yum.conf
# PERL5LIB=/usr/lib/perl perl -MNCM::Component::FreeIPA::CLI -w -e install -- --realm MY.REALM --primary myhost.example.com --domain com --fqdn myhost.example.com --hostcert 1 --otp 'superse\$cret\$OTP'
# 
# echo 'end freeipa post_reboot'
# '
#     doesn't match '(?^m:^yum -y install ncm-freeipa-\d+\.\d+\.\d+-\w+ nss-pam-ldapd ipa-client nss-tools openssl pam_krb5 -c /tmp/aii/yum/yum.conf$)'
# Looks like you failed 1 test of 7.
# INFO: /quattor_test/ncm-freeipa/target/test/cache/aii_2dbasic/profile.1/ccm-active-profile.1-16157.writetest status is mode=420
# INFO: /quattor_test/ncm-freeipa/target/test/cache/aii_2dbasic/profile.1/ccm-active-profile.1-16157 status is mode=420
# INFO: /quattor_test/ncm-freeipa/target/test/cache/aii_2dbasic/latest.cid status is mode=420
# INFO: /quattor_test/ncm-freeipa/target/test/profiles/aii-basic.json status is mode=420
# INFO: /quattor_test/ncm-freeipa/target/test/cache/aii_2dbasic/ccm.cfg status is mode=420
# INFO: /quattor_test/ncm-freeipa/target/test/cache/aii_2dbasic/global.lock status is mode=420
src/test/perl/aii-basic.t ....

@jrha jrha modified the milestones: 20.12, 21.1 Dec 17, 2020
@jrha jrha closed this Apr 13, 2021
@jrha jrha reopened this Apr 13, 2021
@jrha jrha modified the milestones: 21.4, 21.8 Apr 27, 2021
@jrha jrha modified the milestones: 21.8, 21.6 May 4, 2021
@jrha jrha modified the milestones: 21.12, 22.2 Dec 17, 2021
@jrha
Copy link
Member

jrha commented Jun 26, 2023

So… some more data-points on ${rpm.release} expansion:

✔️ Maven 3.6.3 on EL9 (OS version)
✔️ Maven 3.5.4 on EL8 (OS version)
✔️ Maven 3.8.8 on EL7 (upstream)
✔️ Maven 3.5.2 on EL7 (upstream)
✔️ Maven 3.5.0 on EL7 (upstream)
✔️ Maven 3.3.9 on EL7 (upstream)
✔️ Maven 3.3.1 on EL7 (upstream)
✔️ Maven 3.2.5 on EL7 (upstream)
✔️ Maven 3.1.1 on EL7 (upstream)
✔️ Maven 3.0.5 on EL7 (upstream)
❌ Maven 3.0.5 on EL7 (OS version)

So the OS supplied version is somehow broken.
In any case I'm tempted to say that 3.5.4 should now be our minimum supported version.

jrha added a commit to jrha/configuration-modules-core that referenced this pull request Jun 26, 2023
This is the same version shipped with EL8, the version shipped with EL7
no longer meets our needs (see discussion in quattor#1403).
jrha
jrha previously approved these changes Jul 3, 2023
@jrha
Copy link
Member

jrha commented Jul 20, 2023

Looking at this further, the tests pass fine in a vanilla environment and the remaining issue seems to be caused by something mvn_test.sh is doing.

@stdweird
Copy link
Member Author

@jrha fixed in last commit: reason is https://github.com/quattor/maven-tools/blob/master/build-profile/pom.xml#L119
we probably never noticed this before, eg the rpm version in the target config.pan was very likely faulty as well

@jrha
Copy link
Member

jrha commented Jul 28, 2023

Huh! Thanks for the detective work!

@jrha jrha merged commit 7a12116 into quattor:master Jul 28, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants