-
Notifications
You must be signed in to change notification settings - Fork 32
RHSTOR-7460: Upgrade packages in both package.json files #2107
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
RHSTOR-7460: Upgrade packages in both package.json files #2107
Conversation
@NIKHITHAVADDEMPUDI: This pull request references RHSTOR-7210 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
c90d818
to
2fbbaaf
Compare
/retitle RHSTOR-7460: Upgrade packages in both package.json files |
@NIKHITHAVADDEMPUDI: This pull request references RHSTOR-7460 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
@NIKHITHAVADDEMPUDI: This pull request references RHSTOR-7460 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
10abecc
to
c94f95a
Compare
package.json
Outdated
"webpack-bundle-analyzer/ws": "^7.5.10", | ||
"webpack-dev-server/express": "^4.21.0" | ||
}, | ||
"resolutions": {}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"resolutions": {}, |
@NIKHITHAVADDEMPUDI sanity is failing, plz check:
|
LGTM other than above couple of comments, cc @bipuladh @alfonsomthd. |
Ok, i will do the above changes, and push again. |
00f318f
to
37d8016
Compare
37d8016
to
dae1269
Compare
@NIKHITHAVADDEMPUDI: This pull request references RHSTOR-7460 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
92f4e78
to
aa1f275
Compare
|
/retest ci/prow/odf-console-e2e-aws |
/test odf-console-e2e-aws |
/retest-required |
/test odf-console-e2e-aws |
3 similar comments
/test odf-console-e2e-aws |
/test odf-console-e2e-aws |
/test odf-console-e2e-aws |
ea4f690
to
4cc3e07
Compare
/test odf-console-e2e-aws |
3 similar comments
/test odf-console-e2e-aws |
/test odf-console-e2e-aws |
/test odf-console-e2e-aws |
/retest |
/test ci/prow/images |
@NIKHITHAVADDEMPUDI: The specified target(s) for
Use In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/test images |
/test odf-console-e2e-aws |
2 similar comments
/test odf-console-e2e-aws |
/test odf-console-e2e-aws |
Remove all the resolutions and upgrade packages, as no vulnerable versions are found by removing resolutions.
18bd38d
to
0f473b9
Compare
/test odf-console-e2e-aws |
1 similar comment
/test odf-console-e2e-aws |
/override ci/prow/odf-console-e2e-aws |
@alfonsomthd: Overrode contexts on behalf of alfonsomthd: ci/prow/odf-console-e2e-aws In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: NIKHITHAVADDEMPUDI, SanjalKatiyar The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
84da487
into
red-hat-storage:master
https://issues.redhat.com/browse/RHSTOR-7210
Remove all the resolutions and upgrade packages, as no vulnerable versions are found by removing resolutions.
The following packages are not upgraded
@openshift-console/dynamic-plugin-sdk: All "{ singleton: true, allowFallback: false }" dependencies in odf-console should have same versions as used by OCP console.
@openshift-console/dynamic-plugin-sdk-internal: All "{ singleton: true, allowFallback: false }" dependencies in odf-console should have same versions as used by OCP console.
@openshift-console/dynamic-plugin-sdk-webpack: This is already in latest stable version. Latest one is a prerelease version.
@patternfly/patternfly:
@patternfly/react-charts:
@patternfly/react-code-editor:
@patternfly/react-core:
@patternfly/react-icons:
@patternfly/react-table:
@patternfly/react-tokens:
@patternfly/react-topology: All "{ singleton: true, allowFallback: false }" dependencies in odf-console should have same versions as used by OCP console.
buffer: This is already in latest version.
cache-loader: This is already in latest version.
circular-dependency-plugin: This is already in latest version.
classnames: This is already in latest version.
css-loader: This is already in latest version.
file-loader: This is already in latest version.
fuzzysearch: This is already in latest version.
i18next-conv: This is already in latest version.
immer: This is already in latest version.
is-ip: This is already in latest version.
js-base64: This is already in latest version.
js-yaml: This is already in latest version.
lodash-es: This is already in latest version.
mobx-react-lite: This is already in latest version.
murmurhash-js: This is already in latest version.
react: All "{ singleton: true, allowFallback: false }" dependencies in odf-console should have same versions as used by OCP console.
react-copy-to-clipboard: This is already in latest version.(Major version is the latest version).
react-dnd: [email protected] uses type: module in its package.json, making its modules ESM (ECMAScript Modules). This requires fully specified imports, like changing import from 'react/jsx-runtime' to import from 'react/jsx-runtime.js'. React 17 does not have corresponding exports in its package.json that map react/jsx-runtime to react/jsx-runtime.js, leading to errors. React 18 defines these exports, so [email protected] works with React 18 but we are using react 17 currently.
react-dnd-html5-backend: react-dnd-html5-backend 16.0.1 was primarily designed for React 18. It is not fully compatible with React 17.
react-dom:
react-dom versions 18 and 19 introduce significant changes that are not fully compatible with React 17.
ReactDOM.render deprecation:
react-dom 18 deprecates ReactDOM.render and ReactDOM.hydrate. Using these methods will make your application behave like it's running React 17.
New Root API:
React 18 introduces a new root API for client rendering using createRoot from react-dom/client. React 17 does not support this API.
Batching:
React 18 introduces automatic batching of state updates, which differs from how React 17 handled batching.
Concurrent Features:
React 18 includes concurrent features, such as Suspense, that are not present in React 17.
react-dropzone: This is already in latest version.
react-helmet: This is already in latest version.
react-i18next: All "{ singleton: true, allowFallback: false }" dependencies in odf-console should have same versions as used by OCP console.
react-linkify: This is already in latest stable version.
react-redux: All "{ singleton: true, allowFallback: false }" dependencies in odf-console should have same versions as used by OCP console.
react-router-dom-v5-compat: All "{ singleton: true, allowFallback: false }" dependencies in odf-console should have same versions as used by OCP console.
redux: All "{ singleton: true, allowFallback: false }" dependencies in odf-console should have same versions as used by OCP console.
resolve-url-loader: This is already in latest version.
sass-loader: This is already in latest version.
style-loader: Trying to resolve, the below error on upgradation
thread-loader: This is already in latest version.
ts-loader: This is already in latest version.
typesafe-actions: This is already in latest version.
victory-core: This is already in latest version.
Webpack: Getting errors on upgradation, trying to resolve
webpack-bundle-analyzer: This is already in latest version.
webpack-cli: This is already in latest version.
yup: Getting errors on upgradation, trying to resolve.
@discoveryjs/json-ext: This is already in latest version.
@testing-library/jest-dom: This is already in latest version.
@testing-library/react-hooks: This is already in latest version.
@testing-library/user-event: This is already in latest version.
@types/jest: This is already in latest version.
@types/lodash-es: This is already in latest version.
@types/react-helmet: This is already in latest version.
@types/react-window: This is already in latest version.
eslint-config-airbnb: This is already in latest version.
eslint-plugin-chai-friendly: This is already in latest version.
eslint-plugin-import: This is already in latest version.
eslint-plugin-jest-dom: This is already in latest version.
eslint-plugin-jsx-a11y: This is already in latest version.
eslint-plugin-mobx: This is already in latest version.
http-server: This is already in latest version.
husky: This is already in latest version.
i18next-parser: This is already in latest version.
identity-obj-proxy: This is already in latest version.
jest: This is already in latest version.
jest-environment-jsdom: This is already in latest version.
jest-silent-reporter: This is already in latest version.
mocha-junit-reporter: This is already in latest version.
mochawesome: This is already in latest version.
stylelint-config-sass-guidelines: This is already in latest version.
Trying to resolves errors with Webpack, style-loader and yup.
eslint-plugin-testing-library and mochawesome-merge are not updated because of
warning [email protected]: The engine "pnpm" appears to be invalid.
error [email protected]: The engine "node" is incompatible with this module. Expected version ">=22". Got "20.16.0"
error Found incompatible module.
below are the error files of webpack, style-loader, yup and google doc link for dependency trees of all removed resolution
webpack5999_errors.txt
style_loader_errors.txt
yup_errors.txt
https://docs.google.com/document/d/1ZGkogGpw2T4-bElSvJ0cO6W2NRFGH5bNMjwwS2Dj4Jw/edit?usp=sharing