# Added a option "mom\_on\_server"=\<boolean\> to existing PTL requirement decorator

**URL:** <https://community.openpbs.org/t/added-a-option-mom-on-server-boolean-to-existing-ptl-requirement-decorator/2469>\
**Category:** Developers\
**Created:** [February 2, 2021, 10:51am UTC](https://community.openpbs.org/t/added-a-option-mom-on-server-boolean-to-existing-ptl-requirement-decorator/2469 "2021-02-02T10:51:12Z")\
**Posts on this page:** 6\
**Page:** 1

<div class="post-metadata">

**Author:** ![birbalsain](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/birbalsain/32/233_2.png) [@birbalsain](https://community.openpbs.org/u/birbalsain)\
**Post date:** [February 2, 2021, 10:51am UTC](https://community.openpbs.org/t/added-a-option-mom-on-server-boolean-to-existing-ptl-requirement-decorator/2469/1 "2021-02-02T10:51:12Z")

</div>

Hi All,  
Please review design document [PBS Pro Confluence](https://openpbs.atlassian.net/wiki/spaces/PD/pages/2497675265/Added+a+option+mom+on+server+boolean+to+existing+requirement+decorator)  
Please do let me know of any comments or suggestions.  
Thanks and Regards,  
Birbal

---

<div class="post-metadata">

**Author:** ![birbalsain](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/birbalsain/32/233_2.png) [@birbalsain](https://community.openpbs.org/u/birbalsain)\
**Post date:** [February 2, 2021, 11:04am UTC](https://community.openpbs.org/t/added-a-option-mom-on-server-boolean-to-existing-ptl-requirement-decorator/2469/2 "2021-02-02T11:04:08Z")

</div>

PTL requirement decorator doesn’t has a option to specify test requirement "mom should be on server.  
Present requirement decorator option “no\_mom\_on\_server” is used only to specify mom should not be on server by setting it’s value to True and it’s by default value is False that mean mom can or can not be on server.

---

<div class="post-metadata">

**Author:** ![visheshh](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/visheshh/32/209_2.png) [@visheshh](https://community.openpbs.org/u/visheshh)\
**Post date:** [February 2, 2021, 11:21am UTC](https://community.openpbs.org/t/added-a-option-mom-on-server-boolean-to-existing-ptl-requirement-decorator/2469/3 "2021-02-02T11:21:44Z")

</div>

Thanks @birbalsain for adding mom\_on\_server requirement to the decorator. Looks good to me.

---

<div class="post-metadata">

**Author:** ![vstumpf](https://avatars.discourse-cdn.com/v4/letter/v/dbc845/32.png) [@vstumpf](https://community.openpbs.org/u/vstumpf)\
**Post date:** [February 3, 2021, 6:18am UTC](https://community.openpbs.org/t/added-a-option-mom-on-server-boolean-to-existing-ptl-requirement-decorator/2469/4 "2021-02-03T06:18:44Z")

</div>

Thanks for the design, looks good to me.

---

<div class="post-metadata">

**Author:** ![lisa-altair](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/lisa-altair/32/643_2.png) [@lisa-altair](https://community.openpbs.org/u/lisa-altair)\
**Post date:** [February 3, 2021, 9:09pm UTC](https://community.openpbs.org/t/added-a-option-mom-on-server-boolean-to-existing-ptl-requirement-decorator/2469/5 "2021-02-03T21:09:51Z")

</div>

Instead of the log message “mom on server” when the test is skipped, can the message please be clearer. After all “mom on server” could mean two different things:

1. skipping the test because there is a mom on the server
2. skipping the test because the test needs a mom on the server

How about “test needs a mom on the server node”?

---

<div class="post-metadata">

**Author:** ![birbalsain](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/birbalsain/32/233_2.png) [@birbalsain](https://community.openpbs.org/u/birbalsain)\
**Post date:** [February 11, 2021, 8:18am UTC](https://community.openpbs.org/t/added-a-option-mom-on-server-boolean-to-existing-ptl-requirement-decorator/2469/6 "2021-02-11T08:18:28Z")

</div>

I think this message is ok because we get skip message as “what requirement is needed”.

1. If requirement is “no\_mom\_on\_server” and doesn’t match then skip message is “no mom on server”  
means no mom on server is needed.  
2.like above , If requirement is “mom\_on\_server” and doesn’t match then skip message is “mom on server”
