forked from netdata/netdata
-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] master from netdata:master #386
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
remove exclusive lock on netdata.lock - needs testing on macos
Reviewer's Guide by SourceryThis pull request introduces a new journal file format (v2) with improved data integrity, enhances the daemon status file with more host and OS information, removes the file locking mechanism, and comments out OS-specific file metadata code. Updated class diagram for journal file v2 formatclassDiagram
class journal_v2_header {
uint32_t magic
usec_t start_time_ut
usec_t end_time_ut
uint32_t extent_count
uint32_t extent_offset
uint32_t metric_count
uint32_t metric_offset
uint32_t page_count
uint32_t page_offset
uint32_t extent_trailer_offset
uint32_t metric_trailer_offset
uint32_t journal_v1_file_size
uint32_t journal_v2_file_size
void *data
}
class journal_extent_list {
uint64_t datafile_offset
uint32_t datafile_size
uint16_t file_index
uint8_t pages
}
class journal_metric_list {
nd_uuid_t uuid
uint32_t entries
uint32_t page_offset
uint32_t delta_start_s
uint32_t delta_end_s
uint32_t update_every_s
}
class journal_page_header {
uint32_t crc
uint32_t uuid_offset
uint32_t entries
nd_uuid_t uuid
}
class journal_page_list {
uint32_t delta_start_s
uint32_t delta_end_s
uint32_t extent_index
uint32_t update_every_s
uint16_t page_length
uint8_t type
}
journal_v2_header -- journal_extent_list : contains
journal_v2_header -- journal_metric_list : contains
journal_metric_list -- journal_page_header : contains
journal_page_header -- journal_page_list : contains
journal_page_list -- journal_extent_list : refers to
note for journal_v2_header "Journal v2 header structure"
note for journal_extent_list "Extent section item"
note for journal_metric_list "Metric section item"
note for journal_page_header "Page section item header"
note for journal_page_list "Page section item"
Updated class diagram for DAEMON_STATUS_FILEclassDiagram
class DAEMON_STATUS_FILE {
OS_SYSTEM_MEMORY memory
OS_SYSTEM_DISK_SPACE var_cache
const char *architecture
const char *virtualization
const char *container
const char *kernel_version
const char *os_name
const char *os_version
const char *os_id
const char *os_id_like
bool read_system_info
}
note for DAEMON_STATUS_FILE "Daemon status file structure"
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.1)
Can you help keep this open source service alive? 💖 Please sponsor : )
Summary by Sourcery
This pull request introduces several enhancements and a new feature focused on improving data integrity and expanding the system information available in the daemon status. It adds CRC32 checksums to journal files for enhanced data integrity and enriches the daemon status file with detailed host and OS information.
New Features:
Enhancements: