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

[c9s] Build failure #23

Open
sandrobonazzola opened this issue Apr 7, 2022 · 3 comments
Open

[c9s] Build failure #23

sandrobonazzola opened this issue Apr 7, 2022 · 3 comments
Labels
help wanted Extra attention is needed ux

Comments

@sandrobonazzola
Copy link
Member

On CentOS Stream 9 the build uses:

================================================================================
 Package                     Arch   Version                     Repo       Size
================================================================================
Installing:
 ovirt-engine-nodejs-modules noarch 2.2.3-1.el9                 copr_base 196 M
Installing dependencies:
 nodejs                      x86_64 1:16.14.0-2.el9~bootstrap   appstream 216 k
 nodejs-libs                 x86_64 1:16.14.0-2.el9~bootstrap   appstream  14 M

Transaction Summary
================================================================================

Build fails with:

++ yarn install --offline --pure-lockfile
yarn install v1.22.11
[1/4] Resolving packages...
[2/4] Fetching packages...
warning [email protected]: Invalid bin field for "url-loader".
info [email protected]: The platform "linux" is incompatible with this module.
info "[email protected]" is an optional dependency and failed compatibility check. Excluding it from installation.
info [email protected]: The platform "linux" is incompatible with this module.
info "[email protected]" is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
[4/4] Building fresh packages...
Done in 15.30s.
++ yarn check
yarn check v1.22.11
info [email protected]: The platform "linux" is incompatible with this module.
info "[email protected]" is an optional dependency and failed compatibility check. Excluding it from installation.
info [email protected]: The platform "linux" is incompatible with this module.
info "[email protected]" is an optional dependency and failed compatibility check. Excluding it from installation.
warning "patternfly#bootstrap-datepicker#jquery@>=1.7.1 <4.0.0" could be deduped from "3.6.0" to "[email protected]"
warning "patternfly#datatables.net-colreorder-bs#jquery@>=1.7" could be deduped from "3.6.0" to "[email protected]"
warning "patternfly#datatables.net-colreorder#jquery@>=1.7" could be deduped from "3.6.0" to "[email protected]"
warning "patternfly#datatables.net-select#jquery@>=1.7" could be deduped from "3.6.0" to "[email protected]"
warning "patternfly#datatables.net#jquery@>=1.7" could be deduped from "3.6.0" to "[email protected]"
warning "patternfly#drmonty-datatables-colvis#jquery@>=1.7.0" could be deduped from "3.6.0" to "[email protected]"
warning "patternfly#patternfly-bootstrap-treeview#jquery@^3.4.1" could be deduped from "3.6.0" to "[email protected]"
warning "patternfly#bootstrap-switch#jquery@>=1.9.0" could be deduped from "3.6.0" to "[email protected]"
warning "patternfly#eonasdan-bootstrap-datetimepicker#jquery@^1.8.3 || ^2.0 || ^3.0" could be deduped from "3.6.0" to "[email protected]"
warning "datatables.net-colreorder-bs#datatables.net-bs#jquery@>=1.7" could be deduped from "3.6.0" to "[email protected]"
error "patternfly#eonasdan-bootstrap-datetimepicker#jquery" not installed
info Found 10 warnings.
error Found 1 errors.
info Visit https://yarnpkg.com/en/docs/cli/check for documentation about this command.
@sandrobonazzola sandrobonazzola added help wanted Extra attention is needed ux labels Apr 7, 2022
@fitiskin
Copy link

fitiskin commented Apr 7, 2022

Maybe switching from yarn check command to --check-files option would help:

NOTE: The command yarn check has been historically buggy and undermaintained and, as such, has been deprecated and will be removed in Yarn 2.0. You should use yarn install --check-files instead.

From:

yarn install --offline --pure-lockfile
yarn check

To:

yarn install --offline --pure-lockfile --check-files

I've tried to do it locally and the error disappeared.

@sandrobonazzola
Copy link
Member Author

Thanks @fitiskin , giving it a try

@sjd78
Copy link
Member

sjd78 commented Apr 7, 2022

@sandrobonazzola, in ovirt-engine-nodejs-modules, the results of the yarn check is ignored and will not be the source of build issues on C9S.

Unfortunately, our ovirt web projects will not currently build on C9S because they are on webpack 4. Webpack 4 uses a crypto hash function that comes from openssl 1.x. On stream 9 only openssl 3.x is provided. See oVirt/ovirt-web-ui#1568 and oVirt/ovirt-web-ui#1563 for more details. The fix is to upgrade to webpack 5.

Last we looked, patternfly-react v3 was not compatible with webpack 5. I don't think cockpit-ovirt is using patterfly-react anywhere, so that is one less thing to worry about.

cc: @sgratch , @rszwajko

sjd78 added a commit to oVirt/ovirt-engine-nodejs-modules that referenced this issue Apr 7, 2022
* setup-env: update yarn command line

using `yarn install --offline --pure-lockfile --check-files`
instead of
```
yarn install --offline --pure-lockfile
yarn check
```

as suggested by @fitiskin  in oVirt/cockpit-ovirt#23
for solving build issues on CentOS Stream 9.

Signed-off-by: Sandro Bonazzola <[email protected]>

* Update setup-env.sh.in

Co-authored-by: Scott Dickerson <[email protected]>

Co-authored-by: Scott Dickerson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed ux
Projects
None yet
Development

No branches or pull requests

3 participants