Skip to content

Add the JamfSpecV1 proto#26391

Merged
codingllama merged 3 commits intomasterfrom
codingllama/dt-jamfspec
May 17, 2023
Merged

Add the JamfSpecV1 proto#26391
codingllama merged 3 commits intomasterfrom
codingllama/dt-jamfspec

Conversation

@codingllama
Copy link
Copy Markdown
Contributor

The JamfSpecV1 proto is the basis of the Jamf MDM Service configuration.

Added as a "types" proto so it could be reused by eventual Cloud-based setups.

Deviates from the RFD in the definition of Duration-like fields: the gogo typecasts don't allow us to distinguish zero from nil, so we are equating "zero" to "use the defaults".

Relevant sections:

https://github.com/gravitational/teleport.e/issues/826

@codingllama codingllama requested a review from strideynet May 16, 2023 19:55
@codingllama
Copy link
Copy Markdown
Contributor Author

FYI @sshahcodes.

@flyinghermit
Copy link
Copy Markdown
Contributor

Thanks @codingllama. This will be handy replacing my test protos: #26374

@codingllama codingllama force-pushed the codingllama/dt-jamfspec branch from f64246f to 8475bbc Compare May 17, 2023 14:30
@codingllama
Copy link
Copy Markdown
Contributor Author

Thanks for the quick reviews, folks.

@codingllama codingllama enabled auto-merge May 17, 2023 14:31
@codingllama codingllama added this pull request to the merge queue May 17, 2023
Merged via the queue into master with commit 72c9c5f May 17, 2023
@codingllama codingllama deleted the codingllama/dt-jamfspec branch May 17, 2023 15:04
@public-teleport-github-review-bot
Copy link
Copy Markdown

@codingllama See the table below for backport results.

Branch Result
branch/v13 Failed

codingllama added a commit that referenced this pull request May 18, 2023
* Add JamfSpecV1 proto

* Correction: filter_rsql is not required

* Update generated protos
codingllama added a commit that referenced this pull request May 18, 2023
* Add JamfSpecV1 proto

* Correction: filter_rsql is not required

* Update generated protos
@r0mant r0mant mentioned this pull request Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants