-
Notifications
You must be signed in to change notification settings - Fork 63
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
remove state from broker/artemis controller #184
remove state from broker/artemis controller #184
Conversation
fyi #185 prevents the removing of state from the address controller. |
905da14
to
2dd3263
Compare
going to sort that security update. but please review independent of that! |
2dd3263
to
2ea43f3
Compare
ba4858f
to
7fbf022
Compare
pushed a rebase against main |
9d35446
to
b146b05
Compare
…an be left behind via creation time
…bose option to see all of the events that the operator listens for
a1ed10b
to
2bd4cb1
Compare
2bd4cb1
to
49fc919
Compare
I have re based, test good locally, however I saw this CI fail without good reason, so there is some timing related issue in play. It would be good if somone could try and reproduce a local failure. The ci just does make test. |
will try that on my local |
but didn't see the failure? all passed @gtully |
@gaohoward thanks. I think i have located the problem on the latest failed ci test - and have just sorted it, the eventually needed to include the get. |
No description provided.