Один коміт, а не десяток тематичних, свідомо: теми переплетені в
спільних файлах (store.go, docker-compose.yml, deploy/README.md), і
розділити їх можна було б лише індексуванням шматків. Коміти, які не
збираються, гірші за один великий — тим паче що це рівно той стан, який
перевірявся разом.
ЩО ПРАЦЮЄ НА СТЕНДІ Й ПЕРЕВІРЕНО ТАМ
0058 подієві алерти: syslog, ncm, compliance спрацьовують у мить
події; правило з нереалізованим джерелом більше не зберігається
мовчки
0059 snmp.walk і прототипи шаблонів — таблиці з динамічним індексом
описуються шаблоном, а не Go
0060 відкат конфігу: план як різниця, маскування паролів із підписом
плану, обов'язковий контрольний збір, verifying при обриві
0061 кнопки Telegram: довге опитування, авторизація не з callback_data
0062 аудит і архів хостів; тест на AST, що падає на ключі без назви
0063 RLS: три ролі, окремий пул для фонових тактів
0064 строки зберігання даних і сторінка сховища
0065 приймач SNMP-трапів; перевірено справжніми пакетами по дроту,
переклад v1→v2 за RFC 3584 дає правильний OID
0066 ескалації сповіщень
0067 алерт про вичерпання диска
0068 поля заливки конфігу переїхали в каталог профілів
Плюс: 137 тестів вебу з нуля (їх не було взагалі), одинадцять справжніх
вад, знайдених ними й виправлених, і виправлення двох інтеграційних
тестів grpcapi, які мовчки пропускались півтора року.
ЩО ЩЕ НЕ ЗАПУСКАЛОСЬ
netpulse установник: одна команда замість 18 змінних і
593 рядків інструкції
RLS з першого запуску нова інсталяція під політиками одразу;
RLS-EXISTING-INSTALL.md лишається тільки для
старих інсталяцій
.forgejo + CI раннер не зареєстрований
Ці три перевірені компіляцією й міркуванням, але не виконанням.
ГОЛОВНИЙ ВИСНОВОК ДВОХ СЕСІЙ
Зелена перевірка доводить рівно те, що вона перевіряє. Тест ізоляції RLS
був правильний і зелений — і пропустив зламаний вхід, бо перевіряв «чи
не видно чужого», коли зламалось «чи видно своє». Інтеграційні тести
grpcapi були зелені, бо не виконувались. Схема, довідник і протокол
описували те, чого в коді не існувало, і виглядало це як готове.
Тому в кожному завданні цих сесій стояла вимога назвати НЕПОКРИТЕ, а
чотири задачі закінчились не можливістю, а відмовою: правило з
нереалізованим джерелом не зберігається, профіль без команд заливки
каже про це замість мовчазної кнопки, міграція RLS валить сама себе на
таблиці без політики, тест словника аудиту падає на ключі без назви.
Подробиці — HISTORY.md, розділи за 26 і 27 серпня.
144 lines
5.5 KiB
Go
144 lines
5.5 KiB
Go
package ncmx
|
||
|
||
import (
|
||
"bytes"
|
||
"context"
|
||
"crypto/sha256"
|
||
"encoding/json"
|
||
"fmt"
|
||
"strings"
|
||
"time"
|
||
|
||
npv1 "github.com/netpulse/netpulse/gen/go/netpulse/v1"
|
||
)
|
||
|
||
// Виконання довільних команд — той самий сеанс, що й збір конфігу, з
|
||
// однією відмінністю: тут цікавий вивід КОЖНОЇ команди, а не лише
|
||
// останньої.
|
||
//
|
||
// Через це Collect тут не годиться, хоч і робить майже те саме: у ньому
|
||
// вивід підготовчих команд свідомо відкидається, а результат проходить
|
||
// перевірку «схоже на конфіг». Обидва рішення правильні для бекапу й
|
||
// обидва хибні тут: `display cpu` не схожий на конфіг і ніколи не
|
||
// пройде looksLikeConfig, а вивід підготовчої команди — єдине місце, де
|
||
// видно, що консоль так і лишилась посторінковою.
|
||
|
||
// Outcome — результат однієї команди.
|
||
type Outcome struct {
|
||
Command string `json:"command"`
|
||
Output string `json:"output"`
|
||
Error string `json:"error,omitempty"`
|
||
}
|
||
|
||
// CommandReport — тіло, яке їде на сервер замість тексту конфігу.
|
||
//
|
||
// JSON, а не суцільний текст із роздільниками: питання до результату
|
||
// завжди адресне («що відповіла ОЦЯ команда»), і різати спільний потік
|
||
// назад на команди означало б удруге робити роботу, яку ми вже зробили
|
||
// тут — до того ж навгад, бо роздільник може трапитись і у виводі.
|
||
type CommandReport struct {
|
||
Outcomes []Outcome `json:"outcomes"`
|
||
// Чому послідовність обірвалась. Порожнє — усі команди відпрацювали.
|
||
Error string `json:"error,omitempty"`
|
||
}
|
||
|
||
// RunCommands виконує всі команди завдання й повертає вивід кожної.
|
||
//
|
||
// Помилку повертає лише тоді, коли розповідати нічого: не під'єднались,
|
||
// не дочекались привітання. Усе, що сталося після входу, — це вміст
|
||
// звіту, а не відмова: половина виводу теж відповідь, і саме в ній
|
||
// зазвичай видно, чому впала друга половина.
|
||
func RunCommands(ctx context.Context, job *npv1.ConfigJob) (Result, error) {
|
||
start := time.Now()
|
||
|
||
if len(job.GetCommands()) == 0 {
|
||
return Result{}, fmt.Errorf("завдання без жодної команди")
|
||
}
|
||
|
||
promptRe, err := compilePrompt(job.GetPromptRegex())
|
||
if err != nil {
|
||
return Result{}, err
|
||
}
|
||
|
||
timeout := job.GetTimeout().AsDuration()
|
||
if timeout <= 0 {
|
||
timeout = 2 * time.Minute
|
||
}
|
||
ctx, cancel := context.WithTimeout(ctx, timeout)
|
||
defer cancel()
|
||
|
||
cred := job.GetCredential()
|
||
|
||
var transcript *bytes.Buffer
|
||
if job.GetCaptureTranscript() {
|
||
transcript = &bytes.Buffer{}
|
||
}
|
||
|
||
conn, err := Dial(ctx, transportName(job.GetTransport()),
|
||
job.GetDevice().GetAddress(), int(job.GetPort()),
|
||
cred.GetUsername(), password(cred), privateKey(cred),
|
||
connectTimeout(timeout), transcript)
|
||
if err != nil {
|
||
return withTranscript(Result{}, transcript), err
|
||
}
|
||
defer conn.Close()
|
||
|
||
cli := NewCLI(conn, Options{
|
||
PromptRe: promptRe,
|
||
MaxBytes: int(job.GetMaxBytes()),
|
||
Transcript: transcript,
|
||
})
|
||
|
||
if err := cli.WaitPrompt(ctx); err != nil {
|
||
return withTranscript(Result{}, transcript),
|
||
fmt.Errorf("привітання пристрою: %w", err)
|
||
}
|
||
|
||
if job.GetEnableRequired() {
|
||
if err := enable(ctx, cli, cred.GetEnablePassword()); err != nil {
|
||
return withTranscript(Result{}, transcript), err
|
||
}
|
||
}
|
||
|
||
rep := runCommandLoop(ctx, cli, job.GetCommands())
|
||
|
||
body, err := json.Marshal(rep)
|
||
if err != nil {
|
||
return withTranscript(Result{}, transcript), err
|
||
}
|
||
sum := sha256.Sum256(body)
|
||
|
||
res := Result{
|
||
Body: body,
|
||
SHA256: sum[:],
|
||
LineCount: len(rep.Outcomes),
|
||
Duration: time.Since(start),
|
||
}
|
||
return withTranscript(res, transcript), nil
|
||
}
|
||
|
||
// runCommandLoop проганяє команди по черзі й збирає вивід кожної.
|
||
//
|
||
// Окремо від RunCommands, щоб перевірятись без мережі: усе, що варте
|
||
// перевірки, відбувається саме тут, а Dial із живим сокетом у тест не
|
||
// затягнеш.
|
||
func runCommandLoop(ctx context.Context, cli *CLI, commands []string) CommandReport {
|
||
rep := CommandReport{Outcomes: make([]Outcome, 0, len(commands))}
|
||
|
||
for _, cmd := range commands {
|
||
out, err := cli.Run(ctx, cmd)
|
||
oc := Outcome{Command: cmd, Output: strings.TrimRight(out, "\n")}
|
||
if err != nil {
|
||
oc.Error = err.Error()
|
||
rep.Outcomes = append(rep.Outcomes, oc)
|
||
// Далі не йдемо. Не дочекались запрошення — означає, що
|
||
// пристрій ще щось пише або чекає на щось інше; наступна
|
||
// команда пішла б у невідомо який контекст, а її вивід
|
||
// приписався б не тій команді.
|
||
rep.Error = fmt.Sprintf("команда %q: %v", cmd, err)
|
||
break
|
||
}
|
||
rep.Outcomes = append(rep.Outcomes, oc)
|
||
}
|
||
return rep
|
||
}
|