openbmc.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: George Liu <liuxiwei1013@gmail.com>
To: Milton Miller II <miltonm@us.ibm.com>
Cc: OpenBMC Maillist <openbmc@lists.ozlabs.org>,
	Brad Bishop <bradleyb@fuzziesquirrel.com>
Subject: Re: Request new repo for Audit function
Date: Fri, 13 Aug 2021 14:09:37 +0800	[thread overview]
Message-ID: <CANFuQ7D+ihCTnQHuyzatLsu8RBa8gLe0iDed18EAM_sy3aPNYQ@mail.gmail.com> (raw)
In-Reply-To: <OFA5154048.C3010138-ON00258730.00127D40-1628824917257@ibm.com>

On Fri, Aug 13, 2021 at 11:22 AM Milton Miller II <miltonm@us.ibm.com> wrote:
>
>
> Reply below
>
> -------- Original Message --------
> From: George Liu
> Date: Thu, August 12, 2021 7:52 PM -0500
>
>
> Hi Brad:
>
> We have a requirement to implement an Audit function, I found that
> there is a design doc[1], but there is no code to implement it.
>
> Could you create a new repo for Audit function?
> eg: phosphor-audit
>
> [1]: https://github.com/openbmc/docs/blob/master/designs/phosphor-audit.md
>
> thx - George Liu
>
> -------- End of Original Message --------
>
> Hi George.
>
>
> Please update the design with consideration for inclusive naming[2] before submitting code for it, as it uses depreciated terminology in it's external control interface.
>
> I see this design was merged almost 2 years ago before this policy was in place but as you said it hasn't been exposed by the community yet so we won't need to maintain compatibility with prior implementations.

Thanks for your advice.
I will read and update this design doc ASAP :)

>
>
> [2] https://github.com/openbmc/docs/blob/master/CONTRIBUTING.md#inclusive-naming
>
> Thanks,
> Milton
>
>

  reply	other threads:[~2021-08-13  6:10 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-13  3:21 Milton Miller II
2021-08-13  6:09 ` George Liu [this message]
2021-08-13  8:41   ` Ivan Mikhaylov
2021-08-13  9:14     ` George Liu
  -- strict thread matches above, loose matches on Subject: below --
2021-08-13  0:51 George Liu
2021-08-16 14:25 ` Patrick Williams
2021-08-17  0:51   ` George Liu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CANFuQ7D+ihCTnQHuyzatLsu8RBa8gLe0iDed18EAM_sy3aPNYQ@mail.gmail.com \
    --to=liuxiwei1013@gmail.com \
    --cc=bradleyb@fuzziesquirrel.com \
    --cc=miltonm@us.ibm.com \
    --cc=openbmc@lists.ozlabs.org \
    --subject='Re: Request new repo for Audit function' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).