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.

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

feat(NODE-1475): Configuration revamp (define config structure and config tool) #1539

Merged
merged 7 commits into from
Sep 18, 2024

Conversation

andrewbattat
Copy link
Member

@andrewbattat andrewbattat commented Sep 17, 2024

NODE-1360

Changes pulled out of: #1190

This PR defines the config structure and config tool, but does not yet integrate the config tool into SetupOS. This PR should not have an impact on the ic-os. Later, the config tool will be utilized for config sanitization, organization, access, and propagation.

@andrewbattat andrewbattat self-assigned this Sep 17, 2024
@github-actions github-actions bot added the feat label Sep 17, 2024
@andrewbattat andrewbattat marked this pull request as ready for review September 17, 2024 21:22
@andrewbattat andrewbattat requested a review from a team as a code owner September 17, 2024 21:22
@github-actions github-actions bot added the @node label Sep 17, 2024
@andrewbattat andrewbattat changed the title feat(NODE-1360): Configuration revamp (define config structure and config tool) feat(NODE-1475): Configuration revamp (define config structure and config tool) Sep 17, 2024
@andrewbattat andrewbattat added this pull request to the merge queue Sep 18, 2024
Merged via the queue into master with commit e880042 Sep 18, 2024
24 checks passed
@andrewbattat andrewbattat deleted the andrew/config-tool branch September 18, 2024 16:30
github-merge-queue bot pushed a commit that referenced this pull request Dec 4, 2024
NODE-1493

update-config.service updates the old ic-os config format to the new
ic-os config format: #1539

Once all node have an updated version of the config, we will be able to
switch nodes to _[actually
use](#1563 the new config format.
Then, once stable, we will delete update-config.service

I've successfully tested on farm and bare metal 🥳

---------

Co-authored-by: IDX GitHub Automation <infra+github-automation@dfinity.org>
github-merge-queue bot pushed a commit that referenced this pull request Jan 8, 2025
NODE-1360


The IC-OS tool has been created, but not yet used by the IC-OS:
#1539

This PR integrates the config tool into SetupOS. The config tool is
utilized for config sanitization, organization, and access.

Note that the *old* config is still being passed to HostOS, so this PR
should have no impact on HostOS or GuestOS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants