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.

Sign up for GitHub

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

Jump to bottom

Fix graylog dashboard ⚠️ Manual action required #359

Merged
merged 5 commits into from
Sep 20, 2023
Merged

Fix graylog dashboard ⚠️ Manual action required #359

merged 5 commits into from
Sep 20, 2023

Conversation

YuryHrytsuk
Copy link
Collaborator

@YuryHrytsuk YuryHrytsuk commented Sep 19, 2023 •
edited
Loading

  • Remove oSparc Graylog Overview dashboard since it is not used
  • Remove configuration of dashboards in script since we rely on content packs which include dashboard configuration
  • Fix bug causing lost dashboards due to double configuration of dashboards

⚠️ Manual action required:

  • Remove the old oSparc Graylog Overview dashboard manually when deploying this commit

@YuryHrytsuk YuryHrytsuk self-assigned this Sep 19, 2023
@YuryHrytsuk
Copy link
Collaborator Author

YuryHrytsuk commented Sep 19, 2023 •
edited
Loading

Related issue: #360

@YuryHrytsuk YuryHrytsuk added t:bug Something isn't working FAST labels Sep 19, 2023
@YuryHrytsuk YuryHrytsuk added this to the the nameless milestone Sep 19, 2023
mrnicegyu11
mrnicegyu11 approved these changes Sep 20, 2023
View reviewed changes
Copy link
Member

@mrnicegyu11 mrnicegyu11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks good, thanks for removing the unused stuff :--) 💪

@YuryHrytsuk YuryHrytsuk merged commit 5590f18 into ITISFoundation:main Sep 20, 2023
@YuryHrytsuk YuryHrytsuk changed the title Fix graylog dashboard Fix graylog dashboard ⚠️ Manual action required Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GitHK GitHK GitHK approved these changes

@mrnicegyu11 mrnicegyu11 mrnicegyu11 approved these changes

@pcrespov pcrespov pcrespov approved these changes

@sanderegg sanderegg sanderegg approved these changes

@matusdrobuliak66 matusdrobuliak66 matusdrobuliak66 approved these changes

@mguidon mguidon Awaiting requested review from mguidon

Assignees

@YuryHrytsuk YuryHrytsuk

Labels
FAST t:bug Something isn't working
Projects
None yet
Milestone
the nameless
Development

Successfully merging this pull request may close these issues.
6 participants
@YuryHrytsuk @GitHK @mrnicegyu11 @pcrespov @sanderegg @matusdrobuliak66