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

EPIC - Upgrade PatternFly from v4 to v5 on kie-tools #468

Open
1 of 3 tasks
Tracked by #1424
tiagobento opened this issue Aug 18, 2023 · 13 comments
Open
1 of 3 tasks
Tracked by #1424

EPIC - Upgrade PatternFly from v4 to v5 on kie-tools #468

tiagobento opened this issue Aug 18, 2023 · 13 comments
Assignees
Labels
area:dependencies Dependency upgrade/update/migration area:tools Issues affecting Apache KIE tooling projects type:non-functional Benchmarks, assessments, reports etc type:tech-debt Things that were left behind an may harm us in the future.

Comments

@yesamer
Copy link

yesamer commented Sep 27, 2023

uniforms-patternfly module has been updated by @lordrip here https://github.com/KaotoIO/kaoto-next/tree/main/packages/uniforms-patternfly
Thank you, Ricardo!

@lordrip
Copy link

lordrip commented Oct 2, 2023

@yesamer, I moved the library to a standalone repository https://github.com/KaotoIO/uniforms-patternfly. The idea is to have independent versioning after each merge to the main branch and also make it easier to bring the library back kie-tools when makes sense.

@jomarko
Copy link

jomarko commented Oct 16, 2023

Hello @lordrip @yesamer @tiagobento , should we do the same for kogito-apps react codebase?

@lordrip
Copy link

lordrip commented Oct 17, 2023

Hi @jomarko, what do you mean by the same?

@jomarko
Copy link

jomarko commented Oct 18, 2023

@lordrip upgrade patternfly dependencies from 4.x to 5.x for kogito-apps

@tiagobento tiagobento added the type:tech-debt Things that were left behind an may harm us in the future. label May 6, 2024
@tiagobento tiagobento changed the title Upgrade PatternFly from 4.x.x to 5.x.x on kie-tools Upgrade PatternFly from 4 to 5 on kie-tools Jun 27, 2024
@tiagobento tiagobento changed the title Upgrade PatternFly from 4 to 5 on kie-tools Upgrade PatternFly from v4 to v5 on kie-tools Jun 27, 2024
@tiagobento
Copy link
Author

For anyone interested on the comments above, everything related to React has been moved from kogito-apps to kie-tools.

@fantonangeli
Copy link

@tiagobento I generated the full list of components to updated on kie-tools. I used the command:
pnpm -rc exec "[ -d 'src' ] && npx @patternfly/pf-codemods@latest ./src || exit 0" >pf-codemods-v5.logs

Here is the list of files
Here is the full logs

@tiagobento
Copy link
Author

@fantonangeli Nice. I guess that's gonna be the way to achieve this... Trying some things to find out what's the best strategy to tackle it. I guess it's fine to upgrade in parts.. we just need to know what packages will be done on each part

@fantonangeli
Copy link

@tiagobento I think we can create a new patternfly-v5-upgrade branch, configuring syncpack to allow Patternfly v4 and v5 together.
Then we upgrade packages by packages or block of packages, depending on how many files needs to be updated.
Then open the final PR to main branch.
WDYT?

@lordrip
Copy link

lordrip commented Sep 28, 2024

Hi @tiagobento @fantonangeli, I think you are aware, but just in case, this is the fork of uniforms-patternfly that we did for Kaoto 👉 https://github.com/KaotoIO/uniforms-patternfly using patternfly v5.

The reason why I mention it is because maybe it could simplify the update process a bit because perhaps you could migrate one application at a time (I don't know if this is something that you're after), despite the application using uniforms, otherwise I guess you would need to upgrade uniforms first. 🤔

@fantonangeli
Copy link

Thank you @lordrip for your very nice work.
I think we can use your uniforms-patternfly to update @kie-tools/uniforms-patternfly

@jomarko
Copy link

jomarko commented Oct 1, 2024

Hi everyone, I would like to understand better the approach we are going to proceed. To me, it sounds we discuss currently these options:

  • upgrade patternfly from v4 to v5 in small steps directly in kie-tools packages, what may bring a problem of parallel usage of different versions?
  • replace kie-tools/uniforms-patternfly with KaotoIO/uniforms-patternfly, I am not sure if this is somehow conforming to kie-tools strategy, I mean there is effort to have kie-tools as one mono repo

thank you for any comment, I am always happy when similar upgrades are achieved

@fantonangeli
Copy link

Hi @jomarko ,

  • upgrade patternfly from v4 to v5 in small steps directly in kie-tools packages, what may bring a problem of parallel usage of different versions?
    We are still defining the groups of packages, possibly isolated, to update, limiting the impact to other groups and how to proceed.

  • replace kie-tools/uniforms-patternfly with KaotoIO/uniforms-patternfly, I am not sure if this is somehow conforming to kie-tools strategy, I mean there is effort to have kie-tools as one mono repo
    I was referring to import their code to kie-tools/packages/uniform-patternfly If it's an "ok way to proceed". Of course we will set again the folders following the kie-tools style.

@fantonangeli fantonangeli changed the title Upgrade PatternFly from v4 to v5 on kie-tools EPIC - Upgrade PatternFly from v4 to v5 on kie-tools Oct 3, 2024
@fantonangeli fantonangeli self-assigned this Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dependencies Dependency upgrade/update/migration area:tools Issues affecting Apache KIE tooling projects type:non-functional Benchmarks, assessments, reports etc type:tech-debt Things that were left behind an may harm us in the future.
Projects
Status: 📋 Backlog
Development

No branches or pull requests

5 participants