-
Notifications
You must be signed in to change notification settings - Fork 60
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
Allow filtering of Node-RED logs when in HA mode #2367
Conversation
Functionally, all looks good. Does need E2E test though, but I'll cover them now @hardillb. Having to pull in |
@joepavitt I've just pushed a tiny update to insert the src markers into the stub drivers fake log output that should help with the tests |
Thanks Ben, was stubbing the API responses with the |
it('display a marker to indicuate with HA replica the logs are from, if present', () => { | ||
// Modify our Instance so that HA is enabled | ||
cy.intercept('GET', '/api/*/projects/*', (req) => { | ||
req.continue((res) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Pezmc TIL about req.continue()
to modify a response in Cypress, rather than mock up a full response, came in real handy here, and flagging in case you'd not seen it before.
part of #2260
Description
Allows filtering of logs for HA instances to only show logs from a single replica
Related Issue(s)
#2260
Checklist
flowforge.yml
?flowforge/helm
to update ConfigMap Templateflowforge/CloudProject
to update values for Staging/ProductionLabels
backport
labelarea:migration
label