search

alces-software / forge-packages

Package descriptors for Alces Flight Forge
0 stars 0 forks source link

Genders is not being patched correctly to set the genders file #12

Closed WilliamMcCumstie closed 5 years ago

WilliamMcCumstie commented 5 years ago

The genders forge package should patch nodeattr to so the genders file can be set with GENDERS_FILE. The issue appears to predate the transition to flight-direct as demonstrated with the 1.22.0+clusterware0 version of genders forge package.

[root@localhost(unconfigured) ~]# fl fo in genders/1.22.0+clusterware0
    Downloading metadata ... Done
Found package: alces/genders version 1.22.0+clusterware0
    Resolving dependencies ... Done
    Downloading alces/genders/1.22.0+clusterware0 ... Done
    Extracting alces/genders/1.22.0+clusterware0 ... Done
    Installing alces/genders/1.22.0+clusterware0 ... Done
     Cleaning up ... Done
[root@localhost(unconfigured) ~]# /opt/flight-direct/opt/genders/bin/nodeattr --expand
nodeattr: /opt/clusterware/etc/genders: error opening genders file
[root@localhost(unconfigured) ~]# export GENDERS_FILE=/tmp/missing
[root@localhost(unconfigured) ~]# /opt/flight-direct/opt/genders/bin/nodeattr --expand
nodeattr: /opt/clusterware/etc/genders: error opening genders file

The patch should be applied in: https://github.com/alces-software/forge-packages/blob/f7a1b8bcca765a89f0c8b36d61e905c529d7295b/tools/genders/package/build.sh#L31