|
Community Feedback Required on PRs 1348 / 1359
Hi all -- As you're aware from recent working group calls we need to develop a mechanism whereby the service configurations can be easily overridden to use Redis for the upcoming Edinburgh release. In
Hi all -- As you're aware from recent working group calls we need to develop a mechanism whereby the service configurations can be easily overridden to use Redis for the upcoming Edinburgh release. In
|
By
Trevor.Conn@...
· #371
·
|
|
Marking Events for CBOR Flow
Hi all -- During today's Core Working Group call as we were discussing various ways for export-distro / app-functions-sdk to notify core-data that an event had been successfully handled, I believe Eri
Hi all -- During today's Core Working Group call as we were discussing various ways for export-distro / app-functions-sdk to notify core-data that an event had been successfully handled, I believe Eri
|
By
Trevor.Conn@...
· #367
·
|
|
[Edgex-tsc-core] Core Contract Model Validators
Just to update everyone on the thread, I just talked with Tony and we’re on the same page. We will be taking the reflection-based approach with the IsValidated() check for complex types. Trevor
Just to update everyone on the thread, I just talked with Tony and we’re on the same page. We will be taking the reflection-based approach with the IsValidated() check for complex types. Trevor
|
By
Trevor.Conn@...
· #366
·
|
|
[Edgex-tsc-core] Core Contract Model Validators
We’re talking in circles now. If you want to have a call, let me know but I have to start working on this. Trevor
We’re talking in circles now. If you want to have a call, let me know but I have to start working on this. Trevor
|
By
Trevor.Conn@...
· #365
·
|
|
[Edgex-tsc-core] Core Contract Model Validators
Hi Tony – An issue with the sample code in your branch is that Addressable must make a call to a.Validate() on line 179 instead of returning nil. This is because there are other endpoints that take an
Hi Tony – An issue with the sample code in your branch is that Addressable must make a call to a.Validate() on line 179 instead of returning nil. This is because there are other endpoints that take an
|
By
Trevor.Conn@...
· #363
·
|
|
[Edgex-tsc-core] Core Contract Model Validators
Thanks for providing that summary Tony. I've been working on incorporating the "is validated" discussion from this morning into the feature branches below. There's a problem in that we have types that
Thanks for providing that summary Tony. I've been working on incorporating the "is validated" discussion from this morning into the feature branches below. There's a problem in that we have types that
|
By
Trevor.Conn@...
· #359
·
|
|
Core Contract Model Validators
Another update on this – This week in the #core channel on Slack Ian Johnson asked me why the approach below didn’t involve creating a Validator interface and using reflection to go through the member
Another update on this – This week in the #core channel on Slack Ian Johnson asked me why the approach below didn’t involve creating a Validator interface and using reflection to go through the member
|
By
Trevor.Conn@...
· #354
·
|
|
Core Contract Model Validators
Hi all -- I wanted to follow up on this discussion we've been having in the Core Working Group about validating our contract models and where that responsibility lies. This past week we talked about t
Hi all -- I wanted to follow up on this discussion we've been having in the Core Working Group about validating our contract models and where that responsibility lies. This past week we talked about t
|
By
Trevor.Conn@...
· #353
·
|
|
JSON Marshal vs Decode
Well, we kind of have a method like that already. https://github.com/edgexfoundry/go-mod-core-contracts/blob/4425df9b51ed055c673fdafb1a97590b6cd0f324/models/operatingstate.go#L53 Again, the problem is
Well, we kind of have a method like that already. https://github.com/edgexfoundry/go-mod-core-contracts/blob/4425df9b51ed055c673fdafb1a97590b6cd0f324/models/operatingstate.go#L53 Again, the problem is
|
By
Trevor.Conn@...
· #351
·
|
|
JSON Marshal vs Decode
I think my hunch on #1 is correct. If I alter the request JSON to include an empty OperatingState, I can step into the unmarshalling of the type now and receive the error I’m looking for. {"_class":"o
I think my hunch on #1 is correct. If I alter the request JSON to include an empty OperatingState, I can step into the unmarshalling of the type now and receive the error I’m looking for. {"_class":"o
|
By
Trevor.Conn@...
· #349
·
|
|
JSON Marshal vs Decode
Lenny – Yes, it does appear that is the mechanism preventing the UnmarshalJSON from being called. But I have two follow ups: Why does it work when calling Marshal()? I suspect it’s because there isn’t
Lenny – Yes, it does appear that is the mechanism preventing the UnmarshalJSON from being called. But I have two follow ups: Why does it work when calling Marshal()? I suspect it’s because there isn’t
|
By
Trevor.Conn@...
· #348
·
|
|
JSON Marshal vs Decode
We may be moving towards a better solution with Ian’s suggestion. I have created a follow up branch here: https://github.com/tsconn23/go-mod-core-contracts/tree/marshal_bug_ian Only the contracts requ
We may be moving towards a better solution with Ian’s suggestion. I have created a follow up branch here: https://github.com/tsconn23/go-mod-core-contracts/tree/marshal_bug_ian Only the contracts requ
|
By
Trevor.Conn@...
· #346
·
|
|
JSON Marshal vs Decode
Hi Ian – See below in blue Trevor
Hi Ian – See below in blue Trevor
|
By
Trevor.Conn@...
· #345
·
|
|
JSON Marshal vs Decode
Whoops – It’s always good to commit one’s changes, not simply add them. They’re there now. Thanks for the catch, Lenny. Trevor
Whoops – It’s always good to commit one’s changes, not simply add them. They’re there now. Thanks for the catch, Lenny. Trevor
|
By
Trevor.Conn@...
· #342
·
|
|
JSON Marshal vs Decode
Hi all – I’m sending this email out as a follow up to today’s Core WG call. The recording of the call can be found on the wiki page and discussion pertinent to this topic starts at about the 30 minute
Hi all – I’m sending this email out as a follow up to today’s Core WG call. The recording of the call can be found on the wiki page and discussion pertinent to this topic starts at about the 30 minute
|
By
Trevor.Conn@...
· #341
·
|
|
[Edgex-tsc-devops] Preliminary Doc on Moving to Go Modules
Hi all -- I'm going to piggy back on the thread below and loop in the Go email distro for more discussion about modules. I've attached a V2 of the previous document that was sent. It includes two more
Hi all -- I'm going to piggy back on the thread below and loop in the Go email distro for more discussion about modules. I've attached a V2 of the previous document that was sent. It includes two more
|
By
Trevor.Conn@...
· #340
·
|
|
Upcoming Go Versioning Enhancements
Dell Customer Communication “In short, it is long past time to add versions to the working vocabulary of both Go developers and our tools, and this proposal describes a way to do that.” https://github
Dell Customer Communication “In short, it is long past time to add versions to the working vocabulary of both Go developers and our tools, and this proposal describes a way to do that.” https://github
|
By
Trevor.Conn@...
· #258
·
|
|
Go Branching/Merging
OK, I see. I will try this for the next feature I take on. Thanks. Trevor Conn Senior Principal Software Engineer Dell Technologies | IoT DellTech Trevor_Conn@Dell.com Round Rock, TX USA _____________
OK, I see. I will try this for the next feature I take on. Thanks. Trevor Conn Senior Principal Software Engineer Dell Technologies | IoT DellTech Trevor_Conn@Dell.com Round Rock, TX USA _____________
|
By
Trevor.Conn@...
· #239
·
|
|
Go Branching/Merging
Dell Customer Communication There’s been a lot of talk recently around how we manage commits and commit messages in GitHub to make the history more clean. While in Portland earlier this week, I descri
Dell Customer Communication There’s been a lot of talk recently around how we manage commits and commit messages in GitHub to make the history more clean. While in Portland earlier this week, I descri
|
By
Trevor.Conn@...
· #237
·
|
|
Regarding Base Functionality in Core Services
Dell Customer Communication Taking the Docker/Moby tree as an example, if you look in "pkg" you'll find packages used throughout the application that don't contain any of the primary functionality of
Dell Customer Communication Taking the Docker/Moby tree as an example, if you look in "pkg" you'll find packages used throughout the application that don't contain any of the primary functionality of
|
By
Trevor.Conn@...
· #231
·
|