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

Add placeholder flag for contract upgrading #17126

Merged
merged 4 commits into from
Jul 18, 2023
Merged

Conversation

simonmaxen-da
Copy link
Contributor

@simonmaxen-da simonmaxen-da commented Jul 13, 2023

Copy link
Contributor

@nickchapman-da nickchapman-da left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work Simon. LGTM :)

@simonmaxen-da simonmaxen-da enabled auto-merge (squash) July 14, 2023 15:17
@@ -20,8 +20,6 @@ import scala.language.implicitConversions

class ConfigSpec extends AsyncWordSpec with Matchers with CantonFixture {

final override protected lazy val devMode: Boolean = true
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can someone explain this for me please.

Copy link
Contributor

@carlpulley-da carlpulley-da Jul 14, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Setting devMode to be true means Canton logging will be kept.

Copy link
Contributor

@carlpulley-da carlpulley-da Jul 14, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, not quite correct (I think it used to be?). devMode now means our patched version of Canton is used.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

devMode now means our patched version of Canton is used.

@carlpulley-da you are confusing with cantonFixtureDebugMode

devMode = true always meant our patched version of Canton is used (+ canton is started in dev mode).
Such tests that use the patched version of Canton need be mark with the tag dev-canton-test in the bazel Build file which was not the case of this ConfigTest.

Taking into account ConfigTest test does not seems to used any dev feature, it seems more natural to remove the devMode = true that to mark it with dev-canton-test

@simonmaxen-da simonmaxen-da merged commit 2b931bf into main Jul 18, 2023
@simonmaxen-da simonmaxen-da deleted the enable-upgrading branch July 18, 2023 09:17
@nickchapman-da
Copy link
Contributor

Woo Hoo. Finally!

@simonmaxen-da
Copy link
Contributor Author

Once canton has been compiled against this version we can re-enable the tests.

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.

4 participants