Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

[MAINTAIN-203] updated the openy_daxko_gxp_syncer module version up to 1.1.1 #2581

Merged
merged 1 commit into from
Nov 9, 2021
Merged

Conversation

aleevas
Copy link
Contributor

@aleevas aleevas commented Nov 8, 2021

Original Issue, this PR is going to fix: MAINTAIN-203

Steps for review

  • the build should be build in correct way
  • log as admin
  • verify that you don't see the 500 error
  • verify that you can visit /admin/openy/integrations/groupex-pro

General checks

Thank you for your contribution!

@gundevel
Copy link
Collaborator

gundevel commented Nov 8, 2021

Can one of the admins verify this patch? Use "o+k to test" or ''t+est this please" for manual build execution.

@hamrant
Copy link
Contributor

hamrant commented Nov 8, 2021

ok to test

@gundevel
Copy link
Collaborator

gundevel commented Nov 8, 2021

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3567/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3567/

@gundevel
Copy link
Collaborator

gundevel commented Nov 8, 2021

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3567/

@hamrant
Copy link
Contributor

hamrant commented Nov 8, 2021

http://install.openy.cibox.tools/build3567/user/1

Screenshot from 2021-11-08 19-14-37

Standard installation without demo content, @aleevas maybe it not related to your previous fixes and to current PR, but we should find the reason of this issue.

@aleevas can you test on local Open Y Installation Wizard and install Standard Open Y profile without demo content.

@podarok can you provide the error log from this build, maybe now we have a different error?

@hamrant hamrant added Five Jars Assigned for resolution PR: Needs Work Unfinished task. Issues still there labels Nov 8, 2021
@podarok
Copy link
Contributor

podarok commented Nov 8, 2021

[Mon Nov 08 17:27:13.057041 2021] [php7:notice] [pid 523] [client 178.54.63.218:49403] Uncaught PHP Exception Drupal\\Component\\Plugin\\Exception\\PluginNotFoundException: "The "webform" entity type does not exist." at /var/lib/jenkins/jobs/PR_BUILDER_COMPOSER/workspace/build3567/openy-project/docroot/core/lib/Drupal/Core/Entity/EntityTypeManager.php line 143

@hamrant

@hamrant
Copy link
Contributor

hamrant commented Nov 8, 2021

#2581 (comment)

Wow, this is interesting 🤔

@hamrant
Copy link
Contributor

hamrant commented Nov 8, 2021

@podarok looks like @froboy is already faced with this issue - https://www.drupal.org/project/drupal/issues/3056633#comment-14149342 😬

@podarok
Copy link
Contributor

podarok commented Nov 9, 2021

@hamrant ymcatwincities/openy-cibox-build#18

$settings['class_loader_auto_detect'] = FALSE;

@podarok
Copy link
Contributor

podarok commented Nov 9, 2021

retest this please

@gundevel
Copy link
Collaborator

gundevel commented Nov 9, 2021

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3571/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3571/

@gundevel
Copy link
Collaborator

gundevel commented Nov 9, 2021

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3571/

@podarok
Copy link
Contributor

podarok commented Nov 9, 2021

retest this please

@gundevel
Copy link
Collaborator

gundevel commented Nov 9, 2021

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3572/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3572/

@gundevel
Copy link
Collaborator

gundevel commented Nov 9, 2021

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3572/

@podarok
Copy link
Contributor

podarok commented Nov 9, 2021

retest this please

@gundevel
Copy link
Collaborator

gundevel commented Nov 9, 2021

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3573/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3573/

@gundevel
Copy link
Collaborator

gundevel commented Nov 9, 2021

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3573/

@podarok
Copy link
Contributor

podarok commented Nov 9, 2021

image
image

Infrastructure is ready @hamrant

Copy link
Contributor

@podarok podarok left a comment

Choose a reason for hiding this comment

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

code looks good

@gundevel
Copy link
Collaborator

gundevel commented Nov 9, 2021

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3574/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3574/

@gundevel
Copy link
Collaborator

gundevel commented Nov 9, 2021

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3574/

@hamrant hamrant marked this pull request as ready for review November 9, 2021 15:27
@hamrant hamrant added PR: Code Reviewed Code reviewed. Ready for QA PR: Tested Manually tested. Green light for merging and removed PR: Needs Work Unfinished task. Issues still there labels Nov 9, 2021
@podarok podarok merged commit e23a28e into ymcatwincities:9.x-2.x Nov 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Five Jars Assigned for resolution PR: Code Reviewed Code reviewed. Ready for QA PR: Tested Manually tested. Green light for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants