-
Notifications
You must be signed in to change notification settings - Fork 702
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
Fix for Carvel global namespaces #5203
Conversation
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
✅ Deploy Preview for kubeapps-dev canceled.
|
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
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.
Awesome, thanks for giving a last-minute workaround for this problem!
// add the required polyfills (not included in webpack 5) | ||
new NodePolyfillPlugin({ | ||
// Allow using console.log | ||
excludeAliases: ['console'], | ||
}), |
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.
Interesting... don't know how I haven't faced this problem in my local :S
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.
Try removing the modules folder, lock file and yarn install
maybe.
* @param kubeappsConfig Kubeapps configuration | ||
* @returns true if namespace is global | ||
*/ | ||
export function isGlobalNamespace(namespace: string, pluginName: string, kubeappsConfig: IConfig) { |
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.
Great, thanks for the refactor here!
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
Signed-off-by: Rafa Castelblanque rcastelblanq@vmware.com
Description of the change
This PR partially addresses bug #5200 about global namespaces handling for Carvel repositories.
Benefits
It is possible now to have Carvel package repositories correctly created in the global namespace, where global repository depends on the plugin.
Listing of repositories still aggregates both global from Helm and from Carvel.
Possible drawbacks
There is still a lot of frontend code that assumes global namespace is only the one for Helm.
We need to work in the future to refactor the global namespaces part.
Applicable issues
Additional information
Follow-up work tracked in #5202