Skip to content

WIP: Feat/manifest validation - #15

Closed
marcpires wants to merge 23 commits into
lhc:mainfrom
marcpires:feat/manifest-validation
Closed

WIP: Feat/manifest validation#15
marcpires wants to merge 23 commits into
lhc:mainfrom
marcpires:feat/manifest-validation

Conversation

@marcpires

@marcpires marcpires commented Jul 25, 2025

Copy link
Copy Markdown
Collaborator

Descrição

Verifica as Applications do ArgoCD contra o schema adicionado. Inicialmente o esquema da API foi adicionado diretamente neste repo.

Corrige #9

Tipo de mudança

Por favor, exclua as opções que não são relevantes.

  • Correção de bug (mudança não quebradora que resolve um problema)
  • Nova funcionalidade (mudança não quebradora que adiciona funcionalidade)
  • Mudança quebradora (correção ou funcionalidade que causaria a falha da funcionalidade existente)
  • Esta mudança requer uma atualização na documentação

Como Isso Foi Testado?

  1. Instale o act
  2. Altere uma aplicação em apps de forma incorreta
  3. Execute o workflow da raiz do repositório
$ act
  1. O workflow deve quebrar
  2. Volte a aplicação para a configuração correta
  3. Execute novamente o workflow
  4. A pipeline deve passar
  • Adicionada Application configurada incorretamente - quebrou
  • Application atualizada - passou Adicionada
manifest-validation

Para executar o workflow localmente

Configuração de Teste:

  • Hardware:
  • Cadeia de ferramentas:
  • SDK:

Lista de Verificação:

  • Meu código segue as diretrizes de estilo deste projeto
  • Eu realizei uma autoavaliação do meu código
  • Eu comentei meu código, especialmente em áreas difíceis de entender
  • Eu fiz as alterações correspondentes na documentação
  • Minhas alterações não geraram novos avisos
  • Eu adicionei testes que comprovam que minha correção é eficaz ou que minha funcionalidade funciona
  • Os testes de unidade novos e existentes passam localmente com minhas mudanças
  • Quaisquer mudanças dependentes foram mescladas e publicadas nos módulos downstream

marcpires added 17 commits July 23, 2025 07:40
É necessário ter todo o histórico e referencias para as branch, caso o contrário o passo não funcionará como esperado ao ser executado.
Localmente não temos esse problema

Resolves #9
Nos casos em que a versão go Git utilizada for a 2.50, ao se checar as diferenças entre as ramificações obteremos um erro, mesmo utilizando  no checkout
@marcpires marcpires self-assigned this Jul 25, 2025
@marcpires marcpires added the enhancement New feature or request label Jul 25, 2025
@marcpires
marcpires requested a review from manasouza July 25, 2025 13:05
@marcpires
marcpires marked this pull request as draft July 25, 2025 13:06
EstevesDouglas
EstevesDouglas previously approved these changes Aug 11, 2025
@marcpires

Copy link
Copy Markdown
Collaborator Author

@EstevesDouglas O CI ainda não está ok para aprovação. Fiz uma alteração que quebrou o job e tenho que ajustar.

@EstevesDouglas

Copy link
Copy Markdown
Member

Beleza

@EstevesDouglas
EstevesDouglas marked this pull request as ready for review November 26, 2025 22:00
@EstevesDouglas
EstevesDouglas removed the request for review from manasouza November 26, 2025 22:10
@marcpires
marcpires marked this pull request as draft December 14, 2025 09:59
@marcpires

marcpires commented Dec 14, 2025

Copy link
Copy Markdown
Collaborator Author

TODO: Mover o esquema outro repo

@marcpires marcpires changed the title Feat/manifest validation WIP: Feat/manifest validation Dec 16, 2025

@marcpires marcpires left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Remove kyverno app definition
Move ArgoCD schema to own repository

@marcpires

Copy link
Copy Markdown
Collaborator Author

Fechando o PR

@marcpires marcpires closed this Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants