# Proposal to make PBS server set necessary attributes to their default value when these attributes are unset

**URL:** <https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238>\
**Category:** Developers\
**Created:** [October 15, 2018, 11:45pm UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238 "2018-10-15T23:45:19Z")\
**Posts on this page:** 13\
**Page:** 2

<div class="post-metadata">

**Author:** ![anamika](https://avatars.discourse-cdn.com/v4/letter/a/67e7ee/32.png) [@anamika](https://community.openpbs.org/u/anamika)\
**Post date:** [December 8, 2018, 6:59am UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/21 "2018-12-08T06:59:31Z")

</div>

Hey @arungrover, I do not have a strong preference for setting default\_queue from server objects vs pbs\_habitat though it kind of make sense to move it there since you are moving other attributes too so why leaving this one behind.  
I am just unclear how do we unset the default\_queue or mail\_from attribute

---

<div class="post-metadata">

**Author:** ![agrawalravi90](https://avatars.discourse-cdn.com/v4/letter/a/848f3c/32.png) [@agrawalravi90](https://community.openpbs.org/u/agrawalravi90)\
**Post date:** [December 10, 2018, 9:42am UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/22 "2018-12-10T09:42:42Z")

</div>

Thanks for creating the document Arun. Are the tables different from the ones that we have in the reference guide? Are there any attributes who’s default values are being changed from those that are listed in the ref guide?

---

<div class="post-metadata">

**Author:** ![arungrover](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/arungrover/32/40_2.png) [@arungrover](https://community.openpbs.org/u/arungrover)\
**Post date:** [December 10, 2018, 7:11pm UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/23 "2018-12-10T19:11:53Z")

</div>

Thanks @anamika and @agrawalravi90 for reviewing the document.

@agrawalravi90 There are 4 attributes that are changing their defaults “default\_chunk.ncpus, scheduling, log\_events and resources\_default.ncpus”, their default values are listed in the document and color coded in BLUE.

@anamika I can move the queue creation and default\_queue to the server but I am not certain that they are related to PBS server object and that server cannot operate without them. All the attributes I have moved are the ones which are needed by the server to operate normally. “default\_queue”, IMO is not a mandatory attribute that is needed by the server to operate, A user can be enforced to specify a queue during his/her job submission and the server will honor it as long as the queue is valid. In case of mail\_from option, server treats the option as ‘adm’ even when it is unset.

---

<div class="post-metadata">

**Author:** ![anamika](https://avatars.discourse-cdn.com/v4/letter/a/67e7ee/32.png) [@anamika](https://community.openpbs.org/u/anamika)\
**Post date:** [December 11, 2018, 4:40am UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/24 "2018-12-11T04:40:35Z")

</div>

Thanks for clarifying Arun. Document looks good to me.

---

<div class="post-metadata">

**Author:** ![agrawalravi90](https://avatars.discourse-cdn.com/v4/letter/a/848f3c/32.png) [@agrawalravi90](https://community.openpbs.org/u/agrawalravi90)\
**Post date:** [December 11, 2018, 8:44am UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/25 "2018-12-11T08:44:39Z")

</div>

Thanks Arun. I think it’s very important for the document to highlight the fact that you are proposing to change the default values for some of the attributes. Can you please mention that explicitly? Maybe even make it bold? Also, right now the document doesn’t mention what the previous default value was for those attributes, can you mention that as well?

---

<div class="post-metadata">

**Author:** ![mkaro](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/mkaro/32/85_2.png) [@mkaro](https://community.openpbs.org/u/mkaro)\
**Post date:** [December 11, 2018, 6:59pm UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/26 "2018-12-11T18:59:42Z")

</div>

Looks good to me. Thanks @arungrover!

---

<div class="post-metadata">

**Author:** ![arungrover](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/arungrover/32/40_2.png) [@arungrover](https://community.openpbs.org/u/arungrover)\
**Post date:** [December 12, 2018, 10:42pm UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/27 "2018-12-12T22:42:47Z")

</div>

Thanks @anamika and @mkaro for reviewing the document.  
@agrawalravi90 I have added a table specifying the attributes that are going to have a changed value when they are unset. Please have a look.

---

<div class="post-metadata">

**Author:** ![agrawalravi90](https://avatars.discourse-cdn.com/v4/letter/a/848f3c/32.png) [@agrawalravi90](https://community.openpbs.org/u/agrawalravi90)\
**Post date:** [December 13, 2018, 6:39am UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/28 "2018-12-13T06:39:31Z")

</div>

Looks good Arun, I sign-off on the design. One more request though: please add a link to the EDD to the first post on this thread, it’s difficult to scroll and find the link to EDD from your comments. Thanks.

---

<div class="post-metadata">

**Author:** ![bhroam](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/bhroam/32/20_2.png) [@bhroam](https://community.openpbs.org/u/bhroam)\
**Post date:** [January 11, 2019, 10:18pm UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/29 "2019-01-11T22:18:28Z")

</div>

After reviewing the code, I have one possible issue with how this works. Right now default\_chunk.ncpus=1 is the default. If you don’t want this, you can unset it. You might have something like memory only vnodes or vnodes for licenses or something like that. I suspect you could set default\_chunk.ncpus=0, but it isn’t quite the same.

Should we leave default\_chunk alone and allow it to be unset?

Bhroam

---

<div class="post-metadata">

**Author:** ![arungrover](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/arungrover/32/40_2.png) [@arungrover](https://community.openpbs.org/u/arungrover)\
**Post date:** [January 15, 2019, 12:53am UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/30 "2019-01-15T00:53:33Z")

</div>

I guess we could leave default chunk untouched. For the use case you mentioned, the new proposal will break it and force default\_chunk. I guess I’ll make the change in the document to reflect that.

I was double checking all the attributes and saw that pbs\_license\_min/pbs\_license\_max is also mentioned in the document as attributes which will revert to their default values when unset. I think that was an oversight from my side and these two attributes should not be set to a default value when unset. @nithinj since you have more knowledge about license related attributes, what do you think?

---

<div class="post-metadata">

**Author:** ![nithinj](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/nithinj/32/45_2.png) [@nithinj](https://community.openpbs.org/u/nithinj)\
**Post date:** [January 16, 2019, 12:40pm UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/31 "2019-01-16T12:40:36Z")

</div>

At present, pbs\_license\_min, pbs\_license\_max and pbs\_license\_linger\_time will reset to default on unset. Your table data is par with this behavior.

---

<div class="post-metadata">

**Author:** ![arungrover](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/arungrover/32/40_2.png) [@arungrover](https://community.openpbs.org/u/arungrover)\
**Post date:** [January 19, 2019, 1:07am UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/32 "2019-01-19T01:07:02Z")

</div>

Thanks for confirming @nithinj  
@bhroam I’ve made the change in the document, please let me know if your views.  
Thanks!

---

<div class="post-metadata">

**Author:** ![bhroam](https://yyz2.discourse-cdn.com/flex030/user_avatar/community.openpbs.org/bhroam/32/20_2.png) [@bhroam](https://community.openpbs.org/u/bhroam)\
**Post date:** [January 19, 2019, 2:37am UTC](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238/33 "2019-01-19T02:37:17Z")

</div>

Hey @arungrover,  
Thanks for making the change. I think if node\_fail\_requeue will reset back to the default of 310 on the next server restart, then it should return to the default when it is unset.

Just a nit, in the table to the right, you say ‘teated’ instead of ‘treated’.

[Previous page](https://community.openpbs.org/t/proposal-to-make-pbs-server-set-necessary-attributes-to-their-default-value-when-these-attributes-are-unset/1238.md?page=1)
