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 annotation to tag an ent field as an additionalSubject #185

Merged
merged 1 commit into from
Jan 29, 2024

Conversation

rizzza
Copy link
Contributor

@rizzza rizzza commented Jan 23, 2024

Prior to the events re-work, we were able to tag an annotation on an ent schema field to be included as an additionalSubject in the event message.

Proposing to bring that back, so any api in the infratographer ecosystem does not have to manually manage additional subjects in the generated event hooks.

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@rizzza rizzza requested a review from a team as a code owner January 23, 2024 16:50
@rizzza rizzza added this pull request to the merge queue Jan 29, 2024
Merged via the queue into infratographer:main with commit bec1b00 Jan 29, 2024
6 checks passed
@rizzza rizzza deleted the additional-msg-subjects branch January 29, 2024 18:53
mikemrm added a commit to infratographer/tenant-api that referenced this pull request Apr 4, 2024
This corrects the test which accounts for updates sending AuthRelationshipRequest events on tenant updates.

This is no longer the case as of infratographer/x v0.4.0.
Introduced in [infratographer/x#185](infratographer/x#185]
relationship events are only sent on Create and Delete operations.

Signed-off-by: Mike Mason <mimason@equinix.com>
mikemrm added a commit to infratographer/tenant-api that referenced this pull request Apr 5, 2024
* Update module go.infratographer.com/x to v0.4.0

* AuthRelationshipRequest counts are no longer sent for updates

This corrects the test which accounts for updates sending AuthRelationshipRequest events on tenant updates.

This is no longer the case as of infratographer/x v0.4.0.
Introduced in [infratographer/x#185](infratographer/x#185]
relationship events are only sent on Create and Delete operations.

Signed-off-by: Mike Mason <mimason@equinix.com>

* upgrade infratographer/x to v0.5.0

Signed-off-by: Mike Mason <mimason@equinix.com>

* fix new ci requirements with go upgrade

Signed-off-by: Mike Mason <mimason@equinix.com>

* regenerate schema

Signed-off-by: Mike Mason <mimason@equinix.com>

* remove go version pinning. issue marked resolved

Signed-off-by: Mike Mason <mimason@equinix.com>

---------

Signed-off-by: Mike Mason <mimason@equinix.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Mike Mason <mimason@equinix.com>
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.

3 participants