metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael Miklavcic <>
Subject [DISCUSS] Dev guideline changes for architecture diagrams
Date Thu, 02 May 2019 00:12:31 GMT
Picking up where things left off in the VOTE thread on the subject,
I'm presenting a revision to my original proposal below. I'd like to get
this signed off on before submitting it for another vote. Otto, picking up
where we left off, let me know if this looks good to you.

I'd like to propose a vote to change our dev guidelines which will clarify
the tooling we use to produce diagrams and share the source files for those
diagrams. The original discuss thread is noted at the end of this email. I
propose the dev guidelines and
PR checklist
changed in the following ways:

   1. Under "1.1  Contributing A Code Change"
      1. Change <<"New features and significant bug fixes should be
      documented in the JIRA and appropriate architecture diagrams should be
      attached.  Major features may require a vote.">> to <<"New features
      and significant bug fixes should be documented in the JIRA. Appropriate
      architecture diagrams should be created in
and committed
      to source control as per section 2.4. Diagrams may be requested of PR
      submitters during review either as documentation or as an aid to the
      reviewer. Major features may also require a vote.">>
   2. Under "2.4 Documentation"
      1. New line item <<"Diagrams - We save architecture diagram source
      files in an xml format rendered by (instructions below). This
      is the free tool of choice that we've agreed  to use for exchanging
      diagrams and their source files in Metron.">>
      2. New line item <<Image and diagram source files belong in
      "<module-name>/images-source" and rendered diagrams and images belong in
      3. New subsection <<"Creating and Modifying Diagrams">>. This section
      would provide basic instructions for downloading source files from
   3. Add a new checkbox item under PR checklist heading "For documentation
   related changes" with the following text
      1. Have you ensured that any documentation diagrams have been
      updated, along with their source files, using See
   4. Here is the Jira for migrating/redoing existing diagrams

   - Original DISCUSS thread -
   - Original VOTE thread -

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message