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.

Sign up for GitHub

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

Jump to bottom

LPS-129959 Do not create one company per test. This solves the OOM error and improve the performance of the test #872

Closed
wants to merge 2 commits into from
Closed

LPS-129959 Do not create one company per test. This solves the OOM error and improve the performance of the test #872

wants to merge 2 commits into from

Conversation

achaparro
Copy link
Owner

No description provided.

@achaparro
LPS-129959 Do not create one company per test. This solves the OOM er…
1422fad 
…ror and improve the performance of the test
@achaparro
LPS-129959 We need to call InitCompany since we have to register the …
2c4162b 
…portlet category for this company so the kaleo portlet can be registered too
@liferay-continuous-integration
Copy link
Collaborator

To conserve resources, the PR Tester does not automatically run for every pull.

If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed.

If your pull was never tested, comment "ci:test" to run the PR Tester for this pull.

@achaparro
Copy link
Owner Author

ci:forward

@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering the following test suites:

  •     ci:test:relevant
  •     ci:test:sf

The pull request will automatically be forwarded to the user brianchandotcom if the following test suites pass:

  •     ci:test:relevant
  •     ci:test:sf
  •     ci:test:stable

@achaparro achaparro changed the title LPS-129959 We need to call InitCompany since we have to register the portlet category for this company so the kaleo portlet can be registered too LPS-129959 Do not create one company per test. This solves the OOM error and improve the performance of the test Apr 16, 2021
@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: a7701997ce065011e14273683fa8a924e8a79432

Sender Branch:

Branch Name: LPS-129959
Branch GIT ID: 2c4162b12b1df0748471220ef209aaf0b4adc207

1 out of 1jobs PASSED
1 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:stable - 9 out of 9 jobs passed

✔️ ci:test:relevant - 18 out of 18 jobs passed in 1 hour 47 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: a7701997ce065011e14273683fa8a924e8a79432

Upstream Comparison:

Branch GIT ID: a7701997ce065011e14273683fa8a924e8a79432
Jenkins Build URL: Acceptance Upstream DXP (master) #1769

ci:test:stable - 9 out of 9 jobs PASSED
9 Successful Jobs:
ci:test:relevant - 18 out of 18 jobs PASSED
18 Successful Jobs:
For more details click here.

@liferay-continuous-integration
Copy link
Collaborator

All required test suite(s) passed.
Forwarding pull request to brianchandotcom.
Console

Closed
@liferay-continuous-integration
Copy link
Collaborator

Pull request has been successfully forwarded to brianchandotcom#100844
Console

@liferay-continuous-integration
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci:test:relevant - success ci:test:sf - success ci:test:stable - success
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@achaparro @liferay-continuous-integration