Как написать код красивее, чем никак?

ну да, что-то вроде такого будет норм

os.system("echo "+"charge: "+charge+"\n"+">> ~/tmp/ups.prom")

но это не человеку читать, а прометеусу, так что вывод не особо важен

Как создать словарь со списком машин без повторений?

PavelMos, про логику построения понял. Но сейчас этот код, если посмотреть логи, последовательно перезаписывает сам себя и в итоге:

{
  "1": {
    "tgbot": {
      "new_name:": "tgbot-go-7.node.eu.consul",
      "os:": "Debian 10.1 x64",
      "memory:": 2048,
      "cpu:": 4,
      "cpuNumber:": 4,
      "disk:": 8192
    }
  },
  "2": {
    "tgbot": {
      "new_name:": "tgbot-go-7.node.eu.consul",
      "os:": "Debian 10.1 x64",
      "memory:": 2048,
      "cpu:": 4,
      "cpuNumber:": 4,
      "disk:": 8192
    }
  },

Есть другой вариант написания:

vm_to_create = {}
nodes = {1:0,2:0,3:0,4:0,5:0}

for vm in asl_vms:
    if check_language(vm['name']) and check_os_version(vm['distribution']['description']):
        first_part = get_service_name(vm['name'])
        new_vm_name = get_next_hostname(vm['name'])
        vm_res = {'first_part_name:' : first_part, 'os:' : vm['distribution']['description'],'memory:' : vm['memory'],'cpu:' : vm['cpu'],'cpuNumber:' : vm['cpuNumber'],'disk:' : vm['disk']}
        for num in nodes:
            if first_part not in nodes.values():
                vm_to_create[new_vm_name] = vm_res
                nodes[num] = vm_to_create
            else:
                continue
print(json.dumps(nodes))

Он работает, но возвращает весь списко машин под каждым идексом node, т.е. моя псевдо проверка: if first_part not in nodes.values(): не работает.

Суть в том, что на каждом индексе node не должно быть повторяющихся имен. Т.Е. вывод должен выглядеть так:

1 node:
aff-java-1.node.eu.consul
tgbot-go-1.node.eu.consul #tg bot не конфликтует по имени, поэтому записывается на 1 node
2 node: 
aff-java-3.node.eu.consul #он не на первом, так как там уже есть aff
3 node: 
0
4 node:
0
5 node: #3,4,5 создаются, но пустые, просто в данном случае машины кончились
0

Моя логика в данном варианте, на каждой итерации проверять, есть ли на таком-то node уже такое-то имя или нет

Как создать словарь со списком машин без повторений?

PavelMos,
по поводу is, ок, спасибо.
ошибка на этой строке:

nodes[num][first_part]=vm_res
TypeError: 'int' object does not support item assignment

Как создать словарь со списком машин без повторений?

а если смещать else, то node остается пустым, я что-то потерялся уже, и не соображаю где мы vm_res обнулили так

if type (nodes[num]) is dict:
	if first_part not in [x.split('-')[0] for x in nodes[num].keys()]:
		nodes[num][first_part]=vm_res
	else:
		nodes[num][first_part]=vm_res

Как создать словарь со списком машин без повторений?

PavelMos, правильно ли я понимаю, что это верная запись кода?

nodes = {1: 0, 2: 0, 3: 0, 4: 0, 5: 0}

for vm in asl_vms:
	if check_language(vm['name']) and check_os_version(vm['distribution']['description']):
		first_part = get_service_name(vm['name'])
		vm_res = {'os:' : vm['distribution']['description'],'memory:' : vm['memory'],'cpu:' : vm['cpu'],'cpuNumber:' : vm['cpuNumber'],'disk:' : vm['disk']}
		for num in nodes:
			if type (nodes[num]) is dict:
				if first_part not in [x.split('-')[0] for x in nodes[num].keys()]:
					nodes[num][first_part]=vm_res
			else:
				nodes[num][first_part]=vm_res
print(json.dumps(nodes))

потому как сейчас ошибка та же, ругается на int в node

TypeError: 'int' object does not support item assignment

Как создать словарь со списком машин без повторений?

PavelMos,

AttributeError: 'int' object has no attribute 'keys'

и вопрос, разве там keys; не к value, мы обращаемся?

Как создать словарь со списком машин без повторений?

DrPalmer, кстати, если сделать так if fst_part in vm_res.values():, то итерация пропуска начинает работать. он видит значения, но.... он пропускает вообще все машины и выводит пустой словарь. почему он вообще все пропустил я вроде понимаю, но как это исправить?

Как создать словарь со списком машин без повторений?

artempawww, не знал, ок, спасибо

Как создать словарь со списком машин без повторений?

PavelMos, если я делаю if re.match(r'aff', list(vm_res.keys())[0]): или if fst_part in vm_res: с любой итерацией словаря vm_res, vm_to_create, ready_for_nodes, он не тригерит continue ни в одном случае. Как в итоге правильно написать это, не понимаю...

Как создать словарь со списком машин без повторений?

artempawww, подскажите, о чем речь

Как создать словарь со списком машин без повторений?

PavelMos, видимо я не понимаю логику цикла, потому что, вывод все равно такой же:
на всех цифрах node, все машины

nodes = {1: 0, 2: 0, 3: 0, 4: 0, 5: 0}

for vm in asl_vms:
	if check_language(vm['name']) == True and check_os_version(vm['distribution']['description']) == True:
		fst_part = get_service_name(vm['name'])
		vm_res = {'1st_part_name:' : fst_part,'os:' : vm['distribution']['description'],'memory:' : vm['memory'],'cpu:' : vm['cpu'],'cpuNumber:' : vm['cpuNumber'],'disk:' : vm['disk']}
		for num in nodes:
			if re.match(r'aff', list(vm_res.keys())[0]):
				continue
			else:
				new_vm_name = get_next_hostname(vm['name'])
				vm_to_create[new_vm_name] = vm_res
				ready_for_nodes[num] = vm_to_create

print(json.dumps(ready_for_nodes))

Как создать словарь со списком машин без повторений?

Dr. Bacon, я знаком с программированием 2 недели, код постепенно превращается из говна и палок в говно и палки с бантиком.

Как создать словарь со списком машин без повторений?

Dr. Bacon, все принты я убирал для форума, да и в целом программы выводит json мне. ПО поводу режет глаза, учусь, поправлю, ок. Но это все не решение проблемы, есть мысли по поводу проблемы, с которой я обращаюсь?

Как можно написать словарь, вложенный в словарь, вложенный в словарь?

Denis Melnikov, можете пожалуйста пояснить, что в n? НУ т.е. нужно чтобы машины последовательно сопоставились с цифрой 1,2,3,4,5 (по кругу) . я что-то не понимаю как бы это работало

Как можно написать словарь, вложенный в словарь, вложенный в словарь?

Denis Melnikov, вы описали 2-ух уровневый массив, который у меня прописан и работает:

vm_res = {'os:' : vm['distribution']['description'],'memory:' : vm['memory'],'cpu:' : vm['cpu'],'cpuNumber:' : vm['cpuNumber'],'disk:' : vm['disk']}
vm_to_create[new_vm_name] = vm_res

А вот как это все вложить в еще один массив цифр от 1 до 5? append вроде в существующий массив добавит данных, а не вложит его, нет?

Почему проверка совпадения не проходит по всем переменным списка?

PavelMos, он вернул not true, даже по тем машин, которые подходят под условие. Мне нужно чтобы цикл взял каждую машину,

welcome-php-2.node.eu.consul
tgbot-go-1.node.eu.consul
aff-java-1337.node.eu.consul

проверил на соответствие

if check_language(vm['name']) == True and check_os_version(vm['distribution']['description']) == True:

и записал в словарь: vm_to_create = Как это сделать?

Почему проверка совпадения не проходит по всем переменным списка?

нет, есть проверка check_language: если элемент списка, приходящего из json, содержит java или go, то true
такая же проверка на debian 11 ,все не Deb11 - True(deb10, Deb9 b тд), соответственно.

Потом по этой проверке

if check_language(vm['name']) == True and check_os_version(vm['distribution']['description']) == True:

, все кто ее прошел, должны быть записаны в новый словарь

Только проверка не работает на все элементы списков, а как записать их в словарь я не знаю

Как правильно зациклить изменения словаря?

PavelMos, есть варианты как в целом этот кусок можно переписать? сейчас он сложный, скорее математика, чем код

Как правильно зациклить изменения словаря?

PavelMos, точно....заработало, убрал ковычки в вызове функции) спасибо!

Как правильно зациклить изменения словаря?

PavelMos, не совсем понял, что имеется в виду. как можно посмотреть?