Skip to content

Conversation

AleYakovlev
Copy link
Collaborator

after this commit we'll see the next changes:
in files:
cfg/config.yaml
cfg/dynconfig.yaml
cfg/names.txt
for example
Module: "SAS-17"

in file
cfg-d/Configure-${cluster}.txt:
Module: "KLG-02"

Copy link

github-actions bot commented Oct 15, 2025

2025-10-15 14:25:50 UTC Pre-commit check linux-x86_64-release-asan for 08f756c has started.
2025-10-15 14:26:04 UTC Artifacts will be uploaded here
2025-10-15 14:28:40 UTC ya make is running...
🟢 2025-10-15 14:30:17 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
22 22 0 0 0 0

🟢 2025-10-15 14:30:20 UTC Build successful.

Copy link

github-actions bot commented Oct 15, 2025

2025-10-15 14:28:54 UTC Pre-commit check linux-x86_64-relwithdebinfo for 08f756c has started.
2025-10-15 14:29:13 UTC Artifacts will be uploaded here
2025-10-15 14:32:19 UTC ya make is running...
🟢 2025-10-15 14:33:40 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
30 30 0 0 0 0

🟢 2025-10-15 14:33:42 UTC Build successful.

Copy link

🟢 2025-10-15 14:29:16 UTC The validation of the Pull Request description is successful.

@AleYakovlev AleYakovlev requested a review from shmel1k October 15, 2025 14:34

def get_module(self, hostname):
queue = self._ask_location(hostname)["location"].get("queue", "")
if not queue:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Я бы ожидал явного требования модуля ( как в datacenter)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
if not queue:
queue = self._ask_location(hostname)["location"]["queue"]
return queue if self._cloud_mode else queue.upper()
node.WalleLocation.Body = int(host.body)
elif self.__cluster_details.use_k8s_api:
node.Location.DataCenter = host.datacenter
node.Location.Module = host.module
Copy link
Collaborator

Choose a reason for hiding this comment

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

Я бы не добавлял в k8s_api пока что

Copy link

github-actions bot commented Oct 15, 2025

2025-10-15 16:01:06 UTC Pre-commit check linux-x86_64-release-asan for e5bb455 has started.
2025-10-15 16:01:19 UTC Artifacts will be uploaded here
2025-10-15 16:03:54 UTC ya make is running...
🟢 2025-10-15 16:05:27 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
19 19 0 0 0 0

🟢 2025-10-15 16:05:30 UTC Build successful.

Copy link

github-actions bot commented Oct 15, 2025

2025-10-15 16:01:07 UTC Pre-commit check linux-x86_64-relwithdebinfo for e5bb455 has started.
2025-10-15 16:01:20 UTC Artifacts will be uploaded here
2025-10-15 16:03:50 UTC ya make is running...
🟢 2025-10-15 16:05:08 UTC Tests successful.

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
27 27 0 0 0 0

🟢 2025-10-15 16:05:11 UTC Build successful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants