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.

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

do not insert NULL for former_identifier and product_type #151

Merged
merged 1 commit into from
Mar 14, 2025

Conversation

ldzhong
Copy link
Contributor

@ldzhong ldzhong commented Mar 12, 2025

The following error messages are reported when running smt-sync command

12sp5:~ # smt-sync refresh
Downloading Product and Repository information
Download (100%)
DBD::mysql::db do failed: Column 'PRODUCT_TYPE' cannot be null at /usr/lib/perl5/vendor_perl/5.18.2/SMT/SCCSync.pm line 892.
DBD::mysql::db do failed: Column 'PRODUCT_TYPE' cannot be null at /usr/lib/perl5/vendor_perl/5.18.2/SMT/SCCSync.pm line 892.

References: bsc#1239441

@mcalmer
Copy link
Member

mcalmer commented Mar 12, 2025

Is product_type being not set a valid product definition?
For me this error looks more like an invalid product definition in SCC.

@ldzhong
Copy link
Contributor Author

ldzhong commented Mar 12, 2025

Yes, the error is caused by a NULL product type definition. I am not familiar with the process, but I guess the product type is not a required option when enabling a new product in SCC. From smt's perspective I think this patch is an enhancement for such case.

@digitaltom
Copy link
Member

Could you please add a note to the changelog?

@ldzhong
Copy link
Contributor Author

ldzhong commented Mar 13, 2025

The changelog has been updated.

@digitaltom
Copy link
Member

I cannot see a change in the changelog (https://github.com/SUSE/smt/blob/SMT12/package/smt.changes). Could you please add a short note there?

This patch is supposed to fix the following error when running `smt-sync refresh` command

> 12sp5:~ # smt-sync refresh
> Downloading Product and Repository information
> Download  (100%)
> DBD::mysql::db do failed: Column 'PRODUCT_TYPE' cannot be null at
  /usr/lib/perl5/vendor_perl/5.18.2/SMT/SCCSync.pm line 892.
> DBD::mysql::db do failed: Column 'PRODUCT_TYPE' cannot be null at
  /usr/lib/perl5/vendor_perl/5.18.2/SMT/SCCSync.pm line 892.

References: bsc#1239441
@ldzhong
Copy link
Contributor Author

ldzhong commented Mar 14, 2025

Sorry for misunderstanding. I have added an entry there.

@digitaltom digitaltom merged commit 968bd5b into SUSE:SMT12 Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants