-
Notifications
You must be signed in to change notification settings - Fork 37
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
upgradeStone fails (referencing seaside.gemstone.com) for a stone built using a custom script #280
Comments
It's worth pointing out that the following produces a stone that does upgrade without errors:
|
The failing upgrade has a project signature of the following for the 3.4.0 stone:
The signature (using the same script) cannot fails in the bootstrapGLASS script, because the same version of Metacello is not present:
...Sooooo, there is a very recent version of Metacello already installed, which is obviously confusing the upgrade script ... |
|
... i was mistaken in thinking that the missing Archive>>close bug was the source of the issue ... I was looking at the state of the image post error and mistakenly thought that the ZipArchive>>close method was missing ... Now that I've (finally) gotten a debugger on the stack, I'm seeing that the failure not related to ZipArchive>>close |
If you look at the upgradeSeasideImage.out log file, you'll see these lines:
That list of configurations is remove from the image prior to reloading metadello and at the end of the day, As a workaround, you should be able to remove the |
Will need to:
|
Apparently my original workaround was not sufficient. After an upgrade the Monticello package is not updated to the correct version (Monticello.v34-dkh.488). If, instead of deleting the ConfigurationOfGofer, the latest version of ConfigurationOfGofer is installed in the pre-upgrade image: Gofer new
url: 'http://seaside.gemtalksystems.com/ss/MetacelloRepository';
package: 'ConfigurationOfGofer';
load.
System commit. The upgrade to 3.5.1 succeeds and indeed the correct version of the Monticello package (using Need to make another run using the latest backup from a test database to confirm that this is a better workaround. |
Using the
The implication of this is that:
|
Hold the phone, it turns out that there is no |
Sooo, the stone To repair and prepare the
|
test comment for slack |
1 similar comment
test comment for slack |
This script is the custom class creation script:
This is the upgradeStone invocation:
Here is the relevant part of the upgradeSeasideImage.log file (killed job during retry of download to produce the stack):
Once final bugfix is determined, will need to:
The text was updated successfully, but these errors were encountered: