🚀 Announcing Beta Release: --dry_run
#1149
Replies: 2 comments 5 replies
-
Love this! I have been integrating this in my CI this week. My use case is the following:
Let me give you some feedback after testing it for some days:
Thanks a lot! |
Beta Was this translation helpful? Give feedback.
-
Hey @kushalshit27 here is some more feedback about the JSON diff that is calculated. I have found what I consider incorrect behavioiur in two cases First case: array order
Check the screenshot to see this clearly: ![]() Ideal behavior for me: The diff in these arrays should not be order-dependant. Second case: Adding new appWhile the above situation seems a minor thing, it gets worse in other situations.
Note: "AIFindr app" is an app that already existed. "LLMFindr app" is the new one: Simulating deployment... The following changes are proposed:
| Resource | Status | Name / Identifier |
|-----------------|----------|----------------------------------|
| Clients | CREATE | LLMFindr app |
| | UPDATE | LLMFindr app | As you see above. Everything seems right, but I don't understand why it creates the new app and also updates it. "clients": [
"Value difference for [AIFindr app.name]: local:LLMFindr app vs remote:AIFindr app",
"Array length difference for [AIFindr app.allowed_logout_urls]: local:1 vs remote:13",
"Array length difference for [AIFindr app.callbacks]: local:1 vs remote:25",
"Array length difference for [AIFindr app.web_origins]: local:1 vs remote:9"
], However, when I saw the JSON details, you can see that it is trying to update the old app with the values of the new app. I'm not sure if this is related to the order again, but It might be. However, this time I didn't find a workaround to avoid it. Maybe this is just a dry-run glitch and the actual import will work without problems, but I don't want to risk it, especially because I would be deploying to prod (and I haven't had the time to create another tenant to test this) I hope all this helps to keep improving the tool. Thanks! |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
🎉 We’re excited to announce the beta release of
--dry_run
support in 8.14.0-beta.0 🚀📘 Learn more about how to use it here: Using Dry Run
Previous Discussions thread
💬 We welcome your feedback on this feature, as your insights are essential to helping us improve and deliver the best possible experience. 🙌
Beta Was this translation helpful? Give feedback.
All reactions