On Wed, Jun 9, 2021 at 10:41 AM Martin Perina <[email protected]> wrote:
> > > On Wed, Jun 9, 2021 at 4:39 PM Liran Rotenberg <[email protected]> > wrote: > >> >> >> On Wed, Jun 9, 2021 at 4:10 PM Martin Perina <[email protected]> wrote: >> >>> >>> >>> On Wed, Jun 9, 2021 at 2:44 PM Milan Zamazal <[email protected]> >>> wrote: >>> >>>> Hi Scott, >>>> >>>> ovirt-engine CI is failing, see >>>> >>>> https://jenkins.ovirt.org/blue/organizations/jenkins/ovirt-engine_standard-check-patch/detail/ovirt-engine_standard-check-patch/12582/pipeline >>>> >>>> The error is: >>>> >>>> UIMessagesTest.doTest:15 approveCertificateTrust does not match the >>>> number of parameters in UIMessages_cs_CZ.properties >>>> >>> >>> Here is the fix, which restores new functionality: >>> >>> https://gerrit.ovirt.org/c/ovirt-engine/+/115170 >>> >> Ah that explains things. The translations were changed manually and were not updated in Zanata. The Zanata pull I did under [1] reimported the old translations. Old translations can still be valid in Zanata because the new source text has not yet been pushed. Only pushing source text from ovirt-engine to Zanata will invalidate translations (in any meaningful way). The translation update scripts I use do not consider any changes to translations in the repo. They consider Zanata to be the source of truth for translations. I will run a zanata push today so any changes to source text made since the last push will invalidate the translations and prevent this issue in future. It can be tricky to keep ovirt-engine source text in sync with Zanata source text and all in sync with the translations. I can probably make the translation changes done in 115170 to Zanata as well so they aren't lost again in the next update. [1] - https://gerrit.ovirt.org/c/ovirt-engine/+/115141 > Thanks Martin! >> I commented in the translation patch as it did pass CI: >> >> Building UI Utils Compatibility (for UICommon) >> >> ... >> >> [INFO] --- maven-surefire-plugin:2.22.0:test (default-test) @ uicompat >> --- [INFO] Tests are skipped. >> >> IMO related to the flags we use in CI: >> make -j1 BUILD_GWT=1 BUILD_ALL_USER_AGENTS=0 BUILD_LOCALES=0 BUILD_UT=0 >> BUILD_VALIDATION=0 >> >> Should we change the make flags in the CI? >> > > Yes, here is the patch: https://gerrit.ovirt.org/c/ovirt-engine/+/115177 > >> >>> I was also curious why CI would pass. The `egrep -q "\.(xml|java)$"` pattern in `check-patch.sh` could just be expanded to `egrep -q "\.(xml|java|properties)$"` and catch translation only patches. > ... >>>> >>>> Indeed, it seems the recently merged translation patch has dropped "{3}" >>>> from approveCertificateTrust. Could you please fix it so that >>>> ovirt-engine CI works again? >>>> >>>> Thanks, >>>> Milan >>>> _______________________________________________ >>>> Devel mailing list -- [email protected] >>>> To unsubscribe send an email to [email protected] >>>> Privacy Statement: https://www.ovirt.org/privacy-policy.html >>>> oVirt Code of Conduct: >>>> https://www.ovirt.org/community/about/community-guidelines/ >>>> List Archives: >>>> https://lists.ovirt.org/archives/list/[email protected]/message/VVSEVCSWSBMVEASPHZBGIVF3EAPQ2KSJ/ >>>> >>> >>> >>> -- >>> Martin Perina >>> Manager, Software Engineering >>> Red Hat Czech s.r.o. >>> _______________________________________________ >>> Devel mailing list -- [email protected] >>> To unsubscribe send an email to [email protected] >>> Privacy Statement: https://www.ovirt.org/privacy-policy.html >>> oVirt Code of Conduct: >>> https://www.ovirt.org/community/about/community-guidelines/ >>> List Archives: >>> https://lists.ovirt.org/archives/list/[email protected]/message/C4YMU4PWCIMJRLRFOJYN3KSKQDCXU4RO/ >>> >> > > -- > Martin Perina > Manager, Software Engineering > Red Hat Czech s.r.o. > -- Scott Dickerson Senior Software Engineer RHV-M Engineering - UX Team Red Hat, Inc
_______________________________________________ Devel mailing list -- [email protected] To unsubscribe send an email to [email protected] Privacy Statement: https://www.ovirt.org/privacy-policy.html oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archives: https://lists.ovirt.org/archives/list/[email protected]/message/B2Y56OS2KPYMCYDJV3LTXJWLQTFQHWSL/
