Update ops/ntp/iosxe: "isconfigured" bool -> str

[jlmcgraw]

• isconfigured is a string, as in nxos
• Update test outputs to match

[ThomasJRyan]

It's difficult to make this change... It's possible that someone out there is relying on this value being a boolean... If this is strictly necessary you might need to look into making a revision of the ops model

[jlmcgraw]

I don't disagree, though I thought that the primary point of these models was that they were consistent across devices and OSes? That is how I came across this: I was expecting to be able to gather data from disparate devices and OSes into identically laid out data structures that I could then use to generate tests from, and that was failing for this "isconfigured" key. I can work around it but was imagining that it would be worth getting aligned If a revision is the way to go, is there a guide for how to do revisions for an OPS model? I see one for parsers <https://github.com/CiscoTestAutomation/getting-started/blob/fabea6b186dc06a9ec4c5af3d88b05ff4bd8d019/develop-guide/writeparser/writeparser.rst#8-revising-a-parser> but have not yet found one for models

…

On Wed, Nov 5, 2025 at 9:54 AM Thomas Ryan ***@***.***> wrote: ***@***.**** requested changes on this pull request. ------------------------------ In pkgs/ops-pkg/src/genie/libs/ops/ntp/iosxe/ntp.py <#196 (comment)> : > @@ -55,7 +55,7 @@ def set_defaults(self, peer, vrf): local_dict = self.info['vrf'][vrf]['associations']['address'][peer]\ ['local_mode'][local_mode].setdefault('isconfigured', {}).\ - setdefault(isconfigured_value, {}) + setdefault(str(isconfigured_value), {}) It's difficult to make this change... It's possible that someone out there is relying on this value being a boolean... If this is strictly necessary you might need to look into making a revision of the ops model — Reply to this email directly, view it on GitHub <#196 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA2IJXXDG2Y6I6LXGPYOYGL33IFSLAVCNFSM6AAAAACLENFC76VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTIMRSGYYDSMJUHE> . You are receiving this because you authored the thread.Message ID: ***@***.***>