What are the Slack Archives?

It’s a history of our time together in the Slack Community! There’s a ton of knowledge in here, so feel free to search through the archives for a possible answer to your question.

Because this space is not active, you won’t be able to create a new post or comment here. If you have a question or want to start a discussion about something, head over to our categories and pick one to post in! You can always refer back to a post from Slack Archives if needed; just copy the link to use it as a reference..

Hi Spryker, here `vendor/spryker/publish-and-synchronize-health-check/src/Spryker/Zed/PublishAndSync

giovanni.piemontese
giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

Hi Spryker,
here vendor/spryker/publish-and-synchronize-health-check/src/Spryker/Zed/PublishAndSynchronizeHealthCheck/Persistence/PublishAndSynchronizeHealthCheckPersistenceFactory.php there is a typo error... Class not found.. wrong use statement.

Thank u!
Fyi: @valerii.trots

Comments

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    Thanks, will check it tomorrow.

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    which class?
    if from this namespace Orm\Zed\PublishAndSynchronizeHealthCheck\ you need to execute propel:install

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    @UKBTMFXMJ did u seen the content of the file???

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    see it right now

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    first use statement...

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    used then in the return of the function..

    the right ORM Class start with Spy...

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    try please to remove all orm class and re-run propel:install.. i dont think that the classes are there (they without Spy at the start of name)

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    that was renamed in xml schema (of course) but the old ones committed & pushed in suite...

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    I see

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    in suite there is no BC (when we run test etc..) but in customer project via composer update then I have a BC

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    thanks @UL65CH0MC!

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    U see.. it happened as said.. the developer at first named the entity without Spy in propel xml schema and generated the orm class.. then he renamed with Spy but he has committed/pushed both and use the old use statement in factory..

    U are welcome!

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    Maybe u can help me to understand another problem that I have with P+S Health Check..

    I executed every project changes as here https://github.com/spryker-shop/suite/pull/798/files but i got an error when i call the health-check in zed

    doc id publish-and-synchronize:search:health-check not found

    I see that in spy_publish_and_synchronize_health_check table is one row written... but _search and _storage tables are empty... and i don't see any event in queue publish.publish_and_synchronize_health_check .

    I missed smth to migrate/update on pyz level?

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    maybe i found it.. i had the job event-trigger-timeout disabled (for testing purpose) in jenkins scheduler and of course the created/updated entities events has not consumed and therefore no events etc...

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    events are now consumed... i have one row written in _search and _storage table... everything was synced but i have the same error when i try to call the zed health check page... I have to investigate in deeper what is the problem...

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    I found @UKBTMFXMJ there is another error

    the key generated in _search table and used as document id in Elastica is publish_and_synchronize_health_check:health-check but here \Spryker\Shared\PublishAndSynchronizeHealthCheckSearch\PublishAndSynchronizeHealthCheckSearchConfig::PUBLISH_AND_SYNCHRONIZE_HEALTH_CHECK_SEARCH_ID is configured publish-and-synchronize:search:health-check

    Ok, i can override on Pyz level, but it should be fixed in core too 😉

  • giovanni.piemontese
    giovanni.piemontese Technical Lead @ Löffelhardt Spryker Solution Partner Posts: 871 🧑🏻‍🚀 - Cadet

    the same happend for storage.. key and check key configured are not the same...