Skip to content

Conversation

@smuzaffar
Copy link
Contributor

No description provided.

@smuzaffar
Copy link
Contributor Author

please test
test will pass here though the class version check fails. Once this is done then I will start PR tests with cms-sw/cmssw-config#89 whcih should fix this issue

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36974/28342

  • This PR adds an extra 8KB to repository

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for master.

It involves the following packages:

  • DataFormats/PatCandidates (reconstruction)

@jpata, @clacaputo, @slava77 can you please review it and eventually sign? Thanks.
@gpetruc, @gouskos, @rovere, @hatakeyamak this is something you requested to watch as well.
@perrotta, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

@smuzaffar
Copy link
Contributor Author

abort

@smuzaffar smuzaffar closed this Feb 15, 2022
@smuzaffar smuzaffar deleted the smuzaffar-patch-4 branch February 15, 2022 18:12
@smuzaffar smuzaffar restored the smuzaffar-patch-4 branch February 16, 2022 09:19
@smuzaffar smuzaffar reopened this Feb 16, 2022
@smuzaffar
Copy link
Contributor Author

please test

@smuzaffar
Copy link
Contributor Author

ok this confirms that there is issue with build rule. I see the following error in build log but build did not fail

error: class 'pat::UserData' has a different checksum for ClassVersion 10. Increment ClassVersion to 11 and assign it to checksum 85383148

cms-sw/cmssw-config#89 should fix this issue. I am restarting tests with cms-sw/cmssw-config#89 now whcih should fail

@smuzaffar
Copy link
Contributor Author

please test with cms-sw/cmssw-config#89

@smuzaffar smuzaffar closed this Mar 7, 2022
@smuzaffar smuzaffar deleted the smuzaffar-patch-4 branch March 7, 2022 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants