Started by upstream project "gerrit-pysim" build number 2,1512151gerrit-pysimjob/gerrit-pysim/COMMENT_TYPEDISTROGERRIT_BRANCHmasterGERRIT_CHANGE_NUMBER39742GERRIT_HOSTgerrit.osmocom.orgGERRIT_PATCHSET_NUMBER5GERRIT_PATCHSET_REVISION3a2cb239bcc6b4dd71107a2b522da581599931b2GERRIT_PATCHSET_UPLOADER_NAMEneelsGERRIT_PORT29418GERRIT_PROJECTpysimGERRIT_REFSPECrefs/changes/42/39742/5GERRIT_REPO_URLssh://jenkins@gerrit.osmocom.org:29418/pysimPIPELINE_BUILD_URLhttps://jenkins.osmocom.org/jenkins/job/gerrit-pysim/2151/PROJECT_NAMEpysimBRANCH_CImaster00007347257347251.06772677207414976770677021423a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2masterssh://jenkins@gerrit.osmocom.org:29418/pysimhttps://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/gerrit-pysim-build/2142/artifacthttps://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/gerrit-pysim-build/changeshttps://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/gerrit-pysim-build/2142/https://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/gerrit-pysim-build/2142/testReportfalse#21427347251309811gerrit-pysim-build #21422142falsetrue21421838567FAILURE1741611676595https://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/gerrit-pysim-build/2142/pySim/esim/saip/personalization.py3a2cb239bcc6b4dd71107a2b522da581599931b21741611631000https://jenkins.osmocom.org/jenkins/user/neelsNeels Hofmeyrnhofmeyr@sysmocom.de[2/7] personalization: refactor ConfigurableParameter, Iccid, Imsi Main points/rationales of the refactoring, details below: 1) common validation implementation 2) offer classmethods The new features are optional, and will be heavily used by batch personalization patches coming soon. Implement Iccid and Imsi to use the new way, with a common abstract DecimalParam implementation. So far leave the other parameter classes working as they always did, to follow suit in subsequent commits. Details: 1) common validation implementation: There are very common validation steps in the various parameter implementations. It is more convenient and much more readable to implement those once and set simple validation parameters per subclass. So there now is a validate_val() classmethod, which subclasses can use as-is to apply the validation parameters -- or subclasses can override their cls.validate_val() for specialized validation. (Those subclasses that this patch doesn't touch still override the self.validate() instance method. Hence they still work as before this patch, but don't use the new common features yet.) 2) offer stateless classmethods: It is useful for... - batch processing of multiple profiles (in upcoming patches) and - user input validation to be able to have classmethods that do what self.validate() and self.apply() do, but do not modify any self.* members. So far the paradigm was to create a class instance to keep state about the value. This remains available, but in addition we make available the paradigm of a singleton that is stateless (the classmethods). Using self.validate() and self.apply() still work the same as before this patch, i.e. via self.input_value and self.value -- but in addition, there are now classmethods that don't touch self.* members. Related: SYS#6768 Change-Id: I6522be4c463e34897ca9bff2309b3706a88b3ce8 2025-03-10 14:00:31 +01003a2cb239bcc6b4dd71107a2b522da581599931b2[2/7] personalization: refactor ConfigurableParameter, Iccid, ImsieditpySim/esim/saip/personalization.pygithttps://jenkins.osmocom.org/jenkins/user/neelsNeels HofmeyrneelsCOMMENT_TYPEDISTROGERRIT_BRANCHmasterGERRIT_CHANGE_NUMBER39742GERRIT_HOSTgerrit.osmocom.orgGERRIT_PATCHSET_NUMBER5GERRIT_PATCHSET_REVISION3a2cb239bcc6b4dd71107a2b522da581599931b2GERRIT_PATCHSET_UPLOADER_NAMEneelsGERRIT_PORT29418GERRIT_PROJECTpysimGERRIT_REFSPECrefs/changes/42/39742/5GERRIT_REPO_URLssh://jenkins@gerrit.osmocom.org:29418/pysimPIPELINE_BUILD_URLhttps://jenkins.osmocom.org/jenkins/job/gerrit-pysim/2151/PROJECT_NAMEpysimBRANCH_CImasterStarted by upstream project "gerrit-pysim-build" build number 2,1422142gerrit-pysim-buildjob/gerrit-pysim-build/0000701570151.00181821423a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2masterssh://jenkins@gerrit.osmocom.org:29418/pysimfalse#214270158804gerrit-pysim-build » distcheck,a1=default,a3=default,a4=default,osmocom-gerrit #21422142falsefalse21421838572SUCCESS1741611676595https://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/gerrit-pysim-build/JOB_TYPE=distcheck,a1=default,a3=default,a4=default,label=osmocom-gerrit/2142/build5-deb12build-ansiblepySim/esim/saip/personalization.py3a2cb239bcc6b4dd71107a2b522da581599931b21741611631000https://jenkins.osmocom.org/jenkins/user/neelsNeels Hofmeyrnhofmeyr@sysmocom.de[2/7] personalization: refactor ConfigurableParameter, Iccid, Imsi Main points/rationales of the refactoring, details below: 1) common validation implementation 2) offer classmethods The new features are optional, and will be heavily used by batch personalization patches coming soon. Implement Iccid and Imsi to use the new way, with a common abstract DecimalParam implementation. So far leave the other parameter classes working as they always did, to follow suit in subsequent commits. Details: 1) common validation implementation: There are very common validation steps in the various parameter implementations. It is more convenient and much more readable to implement those once and set simple validation parameters per subclass. So there now is a validate_val() classmethod, which subclasses can use as-is to apply the validation parameters -- or subclasses can override their cls.validate_val() for specialized validation. (Those subclasses that this patch doesn't touch still override the self.validate() instance method. Hence they still work as before this patch, but don't use the new common features yet.) 2) offer stateless classmethods: It is useful for... - batch processing of multiple profiles (in upcoming patches) and - user input validation to be able to have classmethods that do what self.validate() and self.apply() do, but do not modify any self.* members. So far the paradigm was to create a class instance to keep state about the value. This remains available, but in addition we make available the paradigm of a singleton that is stateless (the classmethods). Using self.validate() and self.apply() still work the same as before this patch, i.e. via self.input_value and self.value -- but in addition, there are now classmethods that don't touch self.* members. Related: SYS#6768 Change-Id: I6522be4c463e34897ca9bff2309b3706a88b3ce8 2025-03-10 14:00:31 +01003a2cb239bcc6b4dd71107a2b522da581599931b2[2/7] personalization: refactor ConfigurableParameter, Iccid, ImsieditpySim/esim/saip/personalization.pygithttps://jenkins.osmocom.org/jenkins/user/neelsNeels HofmeyrCOMMENT_TYPEDISTROGERRIT_BRANCHmasterGERRIT_CHANGE_NUMBER39742GERRIT_HOSTgerrit.osmocom.orgGERRIT_PATCHSET_NUMBER5GERRIT_PATCHSET_REVISION3a2cb239bcc6b4dd71107a2b522da581599931b2GERRIT_PATCHSET_UPLOADER_NAMEneelsGERRIT_PORT29418GERRIT_PROJECTpysimGERRIT_REFSPECrefs/changes/42/39742/5GERRIT_REPO_URLssh://jenkins@gerrit.osmocom.org:29418/pysimPIPELINE_BUILD_URLhttps://jenkins.osmocom.org/jenkins/job/gerrit-pysim/2151/PROJECT_NAMEpysimBRANCH_CImasterStarted by upstream project "gerrit-pysim-build" build number 2,1422142gerrit-pysim-buildjob/gerrit-pysim-build/000011267112671.000021423a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2masterssh://jenkins@gerrit.osmocom.org:29418/pysimfalse#21421126727389gerrit-pysim-build » docs,a1=default,a3=default,a4=default,osmocom-gerrit #21422142falsefalse21421838574SUCCESS1741611676595https://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/gerrit-pysim-build/JOB_TYPE=docs,a1=default,a3=default,a4=default,label=osmocom-gerrit/2142/build5-deb12build-ansiblepySim/esim/saip/personalization.py3a2cb239bcc6b4dd71107a2b522da581599931b21741611631000https://jenkins.osmocom.org/jenkins/user/neelsNeels Hofmeyrnhofmeyr@sysmocom.de[2/7] personalization: refactor ConfigurableParameter, Iccid, Imsi Main points/rationales of the refactoring, details below: 1) common validation implementation 2) offer classmethods The new features are optional, and will be heavily used by batch personalization patches coming soon. Implement Iccid and Imsi to use the new way, with a common abstract DecimalParam implementation. So far leave the other parameter classes working as they always did, to follow suit in subsequent commits. Details: 1) common validation implementation: There are very common validation steps in the various parameter implementations. It is more convenient and much more readable to implement those once and set simple validation parameters per subclass. So there now is a validate_val() classmethod, which subclasses can use as-is to apply the validation parameters -- or subclasses can override their cls.validate_val() for specialized validation. (Those subclasses that this patch doesn't touch still override the self.validate() instance method. Hence they still work as before this patch, but don't use the new common features yet.) 2) offer stateless classmethods: It is useful for... - batch processing of multiple profiles (in upcoming patches) and - user input validation to be able to have classmethods that do what self.validate() and self.apply() do, but do not modify any self.* members. So far the paradigm was to create a class instance to keep state about the value. This remains available, but in addition we make available the paradigm of a singleton that is stateless (the classmethods). Using self.validate() and self.apply() still work the same as before this patch, i.e. via self.input_value and self.value -- but in addition, there are now classmethods that don't touch self.* members. Related: SYS#6768 Change-Id: I6522be4c463e34897ca9bff2309b3706a88b3ce8 2025-03-10 14:00:31 +01003a2cb239bcc6b4dd71107a2b522da581599931b2[2/7] personalization: refactor ConfigurableParameter, Iccid, ImsieditpySim/esim/saip/personalization.pygithttps://jenkins.osmocom.org/jenkins/user/neelsNeels HofmeyrCOMMENT_TYPEDISTROGERRIT_BRANCHmasterGERRIT_CHANGE_NUMBER39742GERRIT_HOSTgerrit.osmocom.orgGERRIT_PATCHSET_NUMBER5GERRIT_PATCHSET_REVISION3a2cb239bcc6b4dd71107a2b522da581599931b2GERRIT_PATCHSET_UPLOADER_NAMEneelsGERRIT_PORT29418GERRIT_PROJECTpysimGERRIT_REFSPECrefs/changes/42/39742/5GERRIT_REPO_URLssh://jenkins@gerrit.osmocom.org:29418/pysimPIPELINE_BUILD_URLhttps://jenkins.osmocom.org/jenkins/job/gerrit-pysim/2151/PROJECT_NAMEpysimBRANCH_CImasterStarted by upstream project "gerrit-pysim-build" build number 2,1422142gerrit-pysim-buildjob/gerrit-pysim-build/000011843118431.000021423a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2masterssh://jenkins@gerrit.osmocom.org:29418/pysimfalse#21421184314849gerrit-pysim-build » pylint,a1=default,a3=default,a4=default,osmocom-gerrit #21422142falsefalse21421838573SUCCESS1741611676595https://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/gerrit-pysim-build/JOB_TYPE=pylint,a1=default,a3=default,a4=default,label=osmocom-gerrit/2142/build4-deb12build-ansiblepySim/esim/saip/personalization.py3a2cb239bcc6b4dd71107a2b522da581599931b21741611631000https://jenkins.osmocom.org/jenkins/user/neelsNeels Hofmeyrnhofmeyr@sysmocom.de[2/7] personalization: refactor ConfigurableParameter, Iccid, Imsi Main points/rationales of the refactoring, details below: 1) common validation implementation 2) offer classmethods The new features are optional, and will be heavily used by batch personalization patches coming soon. Implement Iccid and Imsi to use the new way, with a common abstract DecimalParam implementation. So far leave the other parameter classes working as they always did, to follow suit in subsequent commits. Details: 1) common validation implementation: There are very common validation steps in the various parameter implementations. It is more convenient and much more readable to implement those once and set simple validation parameters per subclass. So there now is a validate_val() classmethod, which subclasses can use as-is to apply the validation parameters -- or subclasses can override their cls.validate_val() for specialized validation. (Those subclasses that this patch doesn't touch still override the self.validate() instance method. Hence they still work as before this patch, but don't use the new common features yet.) 2) offer stateless classmethods: It is useful for... - batch processing of multiple profiles (in upcoming patches) and - user input validation to be able to have classmethods that do what self.validate() and self.apply() do, but do not modify any self.* members. So far the paradigm was to create a class instance to keep state about the value. This remains available, but in addition we make available the paradigm of a singleton that is stateless (the classmethods). Using self.validate() and self.apply() still work the same as before this patch, i.e. via self.input_value and self.value -- but in addition, there are now classmethods that don't touch self.* members. Related: SYS#6768 Change-Id: I6522be4c463e34897ca9bff2309b3706a88b3ce8 2025-03-10 14:00:31 +01003a2cb239bcc6b4dd71107a2b522da581599931b2[2/7] personalization: refactor ConfigurableParameter, Iccid, ImsieditpySim/esim/saip/personalization.pygithttps://jenkins.osmocom.org/jenkins/user/neelsNeels HofmeyrCOMMENT_TYPEDISTROGERRIT_BRANCHmasterGERRIT_CHANGE_NUMBER39742GERRIT_HOSTgerrit.osmocom.orgGERRIT_PATCHSET_NUMBER5GERRIT_PATCHSET_REVISION3a2cb239bcc6b4dd71107a2b522da581599931b2GERRIT_PATCHSET_UPLOADER_NAMEneelsGERRIT_PORT29418GERRIT_PROJECTpysimGERRIT_REFSPECrefs/changes/42/39742/5GERRIT_REPO_URLssh://jenkins@gerrit.osmocom.org:29418/pysimPIPELINE_BUILD_URLhttps://jenkins.osmocom.org/jenkins/job/gerrit-pysim/2151/PROJECT_NAMEpysimBRANCH_CImasterStarted by upstream project "gerrit-pysim-build" build number 2,1422142gerrit-pysim-buildjob/gerrit-pysim-build/00007328827328821.000021423a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2master3a2cb239bcc6b4dd71107a2b522da581599931b23a2cb239bcc6b4dd71107a2b522da581599931b2masterssh://jenkins@gerrit.osmocom.org:29418/pysimfalse#21427328821031283gerrit-pysim-build » test,a1=default,a3=default,a4=default,simtester #21422142falsetrue21421838571FAILURE1741611676595https://jenkins.osmocom.org/jenkins/view/osmo-gsm-tester/job/gerrit-pysim-build/JOB_TYPE=test,a1=default,a3=default,a4=default,label=simtester/2142/simtesterpySim/esim/saip/personalization.py3a2cb239bcc6b4dd71107a2b522da581599931b21741611631000https://jenkins.osmocom.org/jenkins/user/neelsNeels Hofmeyrnhofmeyr@sysmocom.de[2/7] personalization: refactor ConfigurableParameter, Iccid, Imsi Main points/rationales of the refactoring, details below: 1) common validation implementation 2) offer classmethods The new features are optional, and will be heavily used by batch personalization patches coming soon. Implement Iccid and Imsi to use the new way, with a common abstract DecimalParam implementation. So far leave the other parameter classes working as they always did, to follow suit in subsequent commits. Details: 1) common validation implementation: There are very common validation steps in the various parameter implementations. It is more convenient and much more readable to implement those once and set simple validation parameters per subclass. So there now is a validate_val() classmethod, which subclasses can use as-is to apply the validation parameters -- or subclasses can override their cls.validate_val() for specialized validation. (Those subclasses that this patch doesn't touch still override the self.validate() instance method. Hence they still work as before this patch, but don't use the new common features yet.) 2) offer stateless classmethods: It is useful for... - batch processing of multiple profiles (in upcoming patches) and - user input validation to be able to have classmethods that do what self.validate() and self.apply() do, but do not modify any self.* members. So far the paradigm was to create a class instance to keep state about the value. This remains available, but in addition we make available the paradigm of a singleton that is stateless (the classmethods). Using self.validate() and self.apply() still work the same as before this patch, i.e. via self.input_value and self.value -- but in addition, there are now classmethods that don't touch self.* members. Related: SYS#6768 Change-Id: I6522be4c463e34897ca9bff2309b3706a88b3ce8 2025-03-10 14:00:31 +01003a2cb239bcc6b4dd71107a2b522da581599931b2[2/7] personalization: refactor ConfigurableParameter, Iccid, ImsieditpySim/esim/saip/personalization.pygithttps://jenkins.osmocom.org/jenkins/user/neelsNeels Hofmeyr