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

[RHPAM-4889] - adjustments to make it easier to consume from openshift env vars #1607

Merged
merged 3 commits into from
Jul 4, 2024

Conversation

porcelli
Copy link
Member

@porcelli porcelli commented Feb 20, 2024

Thank you for submitting this pull request

JIRA: (please edit the JIRA link if it exists)

RHPAM-4889

referenced Pull Requests: (please edit the URLs of referenced pullrequests if they exist)

  • paste the link(s) from GitHub here
  • link 2
  • link 3 etc.
How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.

build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

How to retest this PR or trigger a specific build:
  • a pull request please add comment: Jenkins retest this

  • a full downstream build please add comment: Jenkins run fdb

  • a compile downstream build please add comment: Jenkins run cdb

  • a full production downstream build please add comment: Jenkins execute product fdb

  • an upstream build please add comment: Jenkins run upstream

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.

If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

@mareknovotny mareknovotny added the rebase-required there are merge conflicts so REBASE label Feb 21, 2024
…roup#1606)

* RHPAM-4889: using a feature flag (jbpm.wb.querymode values DEFAULT or STRICT) to enable/disable strict mode. by default it'll continue behave as before this change.

* RHPAM-4889: reducing duplication.
@porcelli
Copy link
Member Author

@mareknovotny can you help me understanding the checks failure?

@mareknovotny
Copy link
Member

[2024-02-21T23:42:44.605Z] [ERROR] COMPILATION ERROR : 
[2024-02-21T23:42:44.605Z] [INFO] -------------------------------------------------------------
[2024-02-21T23:42:44.605Z] [ERROR] error reading /home/jenkins/.m2/repository/com/google/http-client/google-http-client/1.23.0/google-http-client-1.23.0.jar; zip file is empty
[2024-02-21T23:42:44.605Z] [ERROR] /home/jenkins/workspace/KIE/main/pullrequest/jbpm-wb-main.pr/bc/kiegroup_jbpm-work-items/google-tasks-workitem/target/generated-sources/java/org/jbpm/process/workitem/google/tasks/GetTasksWorkitemHandler.java:[16,1] cannot access org.jbpm.process.workitem.google.tasks
[2024-02-21T23:42:44.605Z]   zip END header not found
[2024-02-21T23:42:44.605Z] [ERROR] /home/jenkins/workspace/KIE/main/pullrequest/jbpm-wb-main.pr/bc/kiegroup_jbpm-work-items/google-tasks-workitem/target/generated-sources/java/org/jbpm/process/workitem/google/tasks/GoogleTasksAuth.java:[34,8] cannot access java.lang
[2024-02-21T23:42:44.605Z]   zip END header not found
[2024-02-21T23:42:44.605Z] [ERROR] /home/jenkins/workspace/KIE/main/pullrequest/jbpm-wb-main.pr/bc/kiegroup_jbpm-work-items/google-tasks-workitem/target/generated-sources/java/org/jbpm/process/workitem/google/tasks/TaskInfo.java:[23,8] cannot access org.jbpm.process.workitem.google
[2024-02-21T23:42:44.605Z]   zip END header not found
[2024-02-21T23:42:44.605Z] [ERROR] /home/jenkins/workspace/KIE/main/pullrequest/jbpm-wb-main.pr/bc/kiegroup_jbpm-work-items/google-tasks-workitem/target/generated-sources/java/org/jbpm/process/workitem/google/tasks/AddTaskWorkitemHandler.java:[56,8] cannot access org.jbpm.process.workitem
[2024-02-21T23:42:44.605Z]   zip END header not found
[2024-02-21T23:42:44.605Z] [INFO] 5 errors 
[2024-02-21T23:42:44.605Z] [INFO] -------------------------------------------------------------
[2024-02-21T23:42:44.605Z] [INFO] Compiling 3 source files to /home/jenkins/workspace/KIE/main/pullrequest/jbpm-wb-main.pr/bc/kiegroup_jbpm-work-items/google-translate-workitem/target/classes
[2024-02-21T23:42:44.605Z] [INFO] -------------------------------------------------------------
[2024-02-21T23:42:44.605Z] [ERROR] COMPILATION ERROR : 
[2024-02-21T23:42:44.605Z] [INFO] -------------------------------------------------------------
[2024-02-21T23:42:44.605Z] [ERROR] error reading /home/jenkins/.m2/repository/com/google/http-client/google-http-client/1.23.0/google-http-client-1.23.0.jar; zip file is empty
[2024-02-21T23:42:44.605Z] [ERROR] /home/jenkins/workspace/KIE/main/pullrequest/jbpm-wb-main.pr/bc/kiegroup_jbpm-work-items/google-translate-workitem/target/generated-sources/java/org/jbpm/process/workitem/google/translate/TranslateWorkitemHandler.java:[16,1] cannot access org.jbpm.process.workitem.google.translate
[2024-02-21T23:42:44.605Z]   zip END header not found
[2024-02-21T23:42:44.605Z] [ERROR] /home/jenkins/workspace/KIE/main/pullrequest/jbpm-wb-main.pr/bc/kiegroup_jbpm-work-items/google-translate-workitem/target/generated-sources/java/org/jbpm/process/workitem/google/translate/DetectLanguageWorkitemHandler.java:[59,8] cannot access java.lang
[2024-02-21T23:42:44.605Z]   zip END header not found
[2024-02-21T23:42:44.605Z] [ERROR] /home/jenkins/workspace/KIE/main/pullrequest/jbpm-wb-main.pr/bc/kiegroup_jbpm-work-items/google-translate-workitem/target/generated-sources/java/org/jbpm/process/workitem/google/translate/GoogleTranslateAuth.java:[21,8] cannot access org.jbpm.process.workitem.google
[2024-02-21T23:42:44.605Z]   zip END header not found
[2024-02-21T23:42:44.605Z] [INFO] 4 errors 
[2024-02-21T23:42:44.605Z] [INFO] -------------------------------------------------------------

these errors come as dependencies are wrongly downloaded imo. Therefore for continued build there are compilation errors.

I can see this error for the first time. We need to try to re-run this to see if it shows again.

jenkins retest this please

@mareknovotny
Copy link
Member

so this CI PR repository is now green, GHA checks have some problems with installing of dependencies from sources, not related. Trigerring FDB to also see downstream is ok with this change.
jenkins do fdb

@porcelli
Copy link
Member Author

@mareknovotny FDB has failed, but doesn't look like it’s related to the changes

@mareknovotny
Copy link
Member

it is not a problem with the PR, but with those artifacts from google which are proxied in internal Nexus Maven Repository, it seems they were downloaded, but not correctly or fully. Therefore those compilation errors and ZIP end headers missing in errors.
trying again FDB and if that is a still problem we can check locally with that Build chain command. I got a msg that the Nexus manager was restarted so it could be that.

@mareknovotny
Copy link
Member

jenkins do fdb

@porcelli
Copy link
Member Author

@mareknovotny thx for the update!

@mareknovotny
Copy link
Member

ok, now FDB green too. Please add backport labels if this need to be on supported branches too 7.67.x and 7.67.x-blue ;)

Copy link

@tomasdavidorg tomasdavidorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, just minor comment.

@mareknovotny mareknovotny changed the title RHPAM-4889: adjustments to make it easier to consume from openshift env vars [RHPAM-4889] - adjustments to make it easier to consume from openshift env vars Mar 26, 2024
@mareknovotny
Copy link
Member

jenkins retest this please

@mareknovotny
Copy link
Member

mareknovotny commented Jun 25, 2024

jenkins do fdb

1 similar comment
@mareknovotny
Copy link
Member

jenkins do fdb

Copy link
Member

@gmunozfe gmunozfe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, well done @porcelli !

@mareknovotny mareknovotny merged commit dc281a4 into kiegroup:main Jul 4, 2024
2 of 4 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 4, 2024
…t env vars (#1607)

* RHPAM-4889: using a feature flag to enable/disable strict mode. (#1606)

* RHPAM-4889: using a feature flag (jbpm.wb.querymode values DEFAULT or STRICT) to enable/disable strict mode. by default it'll continue behave as before this change.

* RHPAM-4889: reducing duplication.

* RHPAM-4889: adjustments to make it easier to consume from openshift env vars

* to solve @tomasdavidorg review comment

---------

Co-authored-by: Marek Novotný <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-7.67.x Generate backport PR for 7.67.x branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants