-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fixes #7638: Rudder agent connect at each runs to its policy server to get its server uuid #932
Conversation
Upmerging this into following branches will be HARD, so please DON'T merge this once the review is OK, but instead signal your approval to me, and I will manually merge and upmerge it at once. |
@ncharles Could you review this please? |
This looks ok, but why isn't the initial promises modified also ? |
Commit modified |
PR rebased |
Do not merge for now, this is screwed up for some reason |
…o get its server uuid
Commit modified |
e031ae8
to
ec4ae42
Compare
Actually, I don't see any changes |
Yeah GitHub seems to be broken at the moment. Give it a few hours... |
ec4ae42
to
8e7517f
Compare
OK, GitHub is OK again. Ready for review again @ncharles |
looks good |
https://www.rudder-project.org/redmine/issues/7638