在OpenStack社區(qū)中，中級貢獻者介于巖石與堅硬的地面之間，他們既不是什么都不懂的菜鳥，也不是因精湛的技能給其他人留下深刻印象的高手。但他們無疑是OpenStack社區(qū)不可或缺的中間力量，中級貢獻者如何提升有效提升自己的貢獻指數(shù)，并且向著成為社區(qū)頂尖高手的目標不斷邁進呢?兩位來自惠普的中級貢獻者代表Scott D’Angelo和Andrea Rosa為我們帶來了一些寶貴的實戰(zhàn)經驗。

[[160731]]

Scott D’Angelo和Andrea Rosa能夠深刻感受到那些“中級”OpenStacker的苦惱。這兩名惠普工程師已經不再是簡單的bug fixer了，他們現(xiàn)在能夠對復雜功能所涉及的東西，以及哪些東西可以被帶來到OpenStack中展開不定期評估。

自OpenStack Folsom版本推出以來就一直從事公有云業(yè)務的高級軟件工程師 D’Angelo 說：“我們的雇主會問，‘嗨，我們想要這個功能，但是這些東西太復雜了。’在這種情況下，你不得不開始制定藍圖和規(guī)范，并且向OpenStack社區(qū)進行反饋。你不得不搞明白這些怎么才能工作得足夠好，甚至需要知道這個功能是否正在被添加。你不得不開始使用一些影響力讓他們關注你的補丁。我們能夠感覺到在新人和關于提供補丁的文檔之間存在著隔閡。”

D’Angelo居住在美國科羅拉多，Rosa居住在英國，但是兩人卻相隔萬里共同合作編寫一個補丁。為了讓Nova擁有一個新的功能，通過合作編寫補丁，兩人對如何展開合作積累了豐富的經驗。通過總結PTL(項目團隊主管)、Core Reviewer(核心評估人員)和同事們正在尋求什么和討厭什么，他們將一些事情推向了新的高度。

而以下便是D’Angelo和Rosa在參與社區(qū)開發(fā)的過程中總結出的8條寶貴經驗。

1. 知道自己要去向何方

與Rosa和D’Angelo溝通的人表示，他們最討厭的是人們不知道這些補丁的歷史。一名Core Reviewer曾經因此被激怒，在他將某個東西標記為“不要這么做，你必須移除這個參數(shù)”之后，在隨后的補丁集中，這一參數(shù)又重新回來，并得到了其他人的批準，且與代碼進行了合并。

D’Angelo說：“通過了解補丁的歷史，你可以知道這些東西為什么會通過這種方式解決，你也可以避免同樣的錯誤。這雖然很乏味，但是在我們的調查中，這是一個反復重復出現(xiàn)的事情。”他表示，我們需要確定哪些工作需要通過IRC與PTL進行溝通。“如果你發(fā)現(xiàn)了項目中一些被忽視的痛點，那么你將會得到鼓勵、歡迎和幫助。”

2. 學習在擁塞的車流中行駛

D’Angelo說：“OpenStack是一個龐大的社區(qū)。這有點像交通阻塞一樣。大家都非常樂于助人，也愿意給予別人鼓勵，但是大家又都非常忙碌。為了把事情做好，你必須盡可能地提高效率，因為每個人的日程都安排得非常滿。有些事情可以加快你的速度，但是有些事情會導致你的速度放緩。”

3. 通過提交信息和單元測試避開不良評估

D’Angelo說：“主動思考你將會遇到的所有問題，避開已知的扣分情況，或是會導致問題的情況。”他稱，自己有時遇到的最大障礙是在單元測試和提交信息上沒有花上足夠的時間就匆忙地提交了代碼。“由于沒有考慮周全，導致整個程序會被打回來，并被扣分。”他說。

這個情況被反復地證明，由于沒有做好補丁功能的解釋工作，糟糕的提交信息只會讓你迅速出局。Rosa稱，不要期望Reviewer會打開漏洞說明書或是介紹，然后完整地閱讀它們。Reviewer需要得到的關于補丁來龍去脈的所有相關信息必須被寫入提交信息中。而解決這一問題的最簡單方法就是使用相關的指南。

D’Angelo承認：“在提交補丁時，提交信息是我最后才編寫的東西。由于沒有在代碼中的提交信息上下功夫，因此會出現(xiàn)一些脫節(jié)。目前我已經知道不能再這么做了。”

關注提交信息的規(guī)則會為我們減少許多麻煩。是讓它們含有72個字符，或者是冒著被扣一分的風險?D’Angelo說：“要減少這些麻煩，至少要知道規(guī)則和提前考慮到在你提交補丁后會有人立即看到它們，并對其進行標記。”

4. 成功通過單元測試

D’Angelo說：“沒有單元測試，你會立即被-1分。它們是我們應該提前規(guī)劃的事情之一。提前寫測試，因為你遲早都要寫。”他還指出，如果先前沒有通過Tox和Pep8，那么測試或補丁將無法通過Jenkins。“我沒有統(tǒng)計過自己看明白了多少提交評估的補丁，但是對于那些不合格的補丁，大家會關注它們失敗的原因。”他說。

5. 避開交通擁堵時間

D’Angelo說：“另一個令人討厭的事情是，有人會強迫或是糾纏Reviewer，讓他們對自己的東西進行評估。在Cinder中，我看到了許多這樣的事情。我們?yōu)檫@一存儲陣列批準了70多個驅動。一直會有新人加入進來，因為他們的公司希望讓一個驅動加入到Cinder中。而這些驅動動不動就包含2000或3000行代碼，這需要很多人花時間進行評估。”

那么有什么訣竅嗎?訣竅就是不要等到到了第二個里程碑版本時才開始讓人對你的補丁進行評估。不要等到在IRC中開始提問，以及不要等到開始加入項目才開始評估。

從Diablo版本開始成為“中級成員”的Rosa對于版本周期也有著相同的見解。“你會向自己的老板承諾許多東西，但是請記住社區(qū)的時間安排與你公司的時間安排有著很大的出入。你需要設定一個期望值，否則你將會像之前的人一樣站在老板面前說‘對不起，我們需要等待下一個周期，時間可能是三個月。’請意識到這一點。”

6.如何應對“交通罰單”?

“如果你被否決了，那么不要灰心。請做好申辯的準備。在你提交補丁之時，請記住扣一分并不意味著世界末日。他們不一定是對的，去找Core Reviewer而不是找普通Reviewer進行申辯。”Rosa說。

向他們清楚地解釋補丁的功能，嘗試著展示它們對項目的提升或是對重大漏洞的彌補情況，或許你的情況會得到反轉。不抱成見，態(tài)度謙虛地提出你的關注點。“我曾經看到一名Core Reviewer將意見由-1修改為了+2的情況。”

7.想改變結果?Review, Review, Review !

D’Angelo說：“你想獲得Review，那么你需要精心準備你的材料，你需要介紹自己的參與情況和學習情況。”

D’Angelo和Rosa建議不要干涉Reviewer的工作，避免人為地讓自己的Stackalytics數(shù)值被扣分。要讓社區(qū)知道自己，包括在IRC中積極表現(xiàn)，這一點也很關鍵。

“Core Reviewer會進行大量的Review工作。他們會看你的綜述，他們會知道你是否進行了出色的總結，你的扣分是否是因為代碼中的錯誤。他們還會知道你被四處扣分的原因是否是因為你文稿上的錯誤。我不認為你真地能夠糊弄到任何人。”

8. 不要為小事?lián)?/strong>