Community: Newbie community pinboard

[Akasurde]

Lot of people ask me same question in lot of meetups and conferences - how we can contribute back to Ansible and where can I find easyfix issue to get started.

We would like to help such people by starting this community pinboard where we can post easyfix or low hanging fruits from the Ansible issues list.

Please feel free to ping other people and add issue which you feel can help newbies/newcomers/want-to-be-contributors.

Here is shortcut link for finding easyfix

Thanks.

[Akasurde]

cc @Devendrabhat @sbulage @digitronik @ptoraskar

[Akasurde]

cc @raghavtan

[debhat]

Thanks, @Akasurde. I'll go through that link.

[raghavtan]

@Akasurde Thanks

[gundalow]

Would love to know how you get on with this.
You may also be interested in HOW TO MAKE YOUR MARK: ANSIBLE COMMUNITY CONTRIBUTIONS
slides & video are linked from this page.

Please do shout out if we can help in anyway. I'm happy to guide/mentor anyone.

[Akasurde]

There is list of open issues segregated by file name - https://ansibullbot.eng.ansible.com/metadata/byfile_sorted.html

[Akasurde]

cc @kylix3511

[Akasurde]

cc @amolkahat

[Akasurde]

cc @akshay196

[akshay196]

Adding @crackcodecamp @ganeshhubale in the thread.

[Akasurde]

cc @sohamdigheoss

[Akasurde]

cc @anis016

[Akasurde]

~Easyfix - Following line needs to be fixed - https://github.com/ansible/ansible/blob/63148b0cc7d396f03215e47ed539145700f32b38/lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py#L1644~

FIXED

[Akasurde]

cc @anurag03

[Akasurde]

cc @Yadnyawalkya

[Akasurde]

~Fix typo just like - https://github.com/ansible/ansible/blob/46fbcf08bc1998db821714ad65a2b271e4e74f89/lib/ansible/modules/network/fortios/fortios_firewall_auth_portal.py#L48~

~Change 'adress' to 'address'. There are ~92 matches in Ansible repo.~

FIXED

[sbulage]

Fix typo just like - https://github.com/ansible/ansible/blob/46fbcf08bc1998db821714ad65a2b271e4e74f89/lib/ansible/modules/network/fortios/fortios_firewall_auth_portal.py#L48

Change 'adress' to 'address'. There are ~92 matches in Ansible repo.

I have fixed it, raising PR

[anis016]

I was almost done! :(

[Akasurde]

@sbulage @anis016 We have PR from @amolkahat https://github.com/ansible/ansible/pull/52143

[felixfontein]

Maybe for the next time, when picking a topic from a source like here (where others might be interested too), announce first when you're starting, so that others can see that someone else is already doing it. That's less frustrating for everyone else :)

[Akasurde]

@felixfontein Good point. There was no issue for this typo so no central place to announce. But going forward we can formalise this. Do you have any easyfix so people here can work on that ? Thanks.

[amolkahat]

@sbulage already raised one: https://github.com/ansible/ansible/pull/52143

CC: @Akasurde

[felixfontein]

@Akasurde Yes, got something: in the FAQ, there are a lot of places where a single backtick is used instead of a double backtick (see source of FAQ). These are formatted in italics instead as code, while most of them propbably should have been the later. This is a common mistake for people more used to Markdown.

I don't know if it also appears in other places in the documentation, or whether all places with a single backtick should be a double backtick, so it is somewhat tedious. There might also other documents ith the same problem.

AFAIR there's no ticket for this.

[Akasurde]

Before starting work on any issue mentioned here or GH issue page, please add a note saying that you are working on it. So that others will be aware and avoid duplicate efforts.

[anis016]

hy @Akasurde, can I work on this issue https://github.com/ansible/ansible/issues/51421 ?

[Akasurde]

@anis016 Sure.

[Akasurde]

Typo fixes - While working on issue, please add a note

• ~https://github.com/ansible/ansible/blob/6a82827dddbe39d478bd148a66a44185ac52a399/lib/ansible/module_utils/facts/network/base.py#L44~
• ~Find 'precendence' and change to 'precedence'~
• ~Find 'arbitary' and change to 'arbitrary'~
• ~Find 'sucessfully' and change to 'successfully'~

[anis016]

I plan to work on the typos! I will submit a PR today.

[Akasurde]

Unsupported argument -

• https://github.com/ansible/ansible/blob/52d8f9c3973a7ad4c8cbc077ba0733e93b4cae22/lib/ansible/modules/database/postgresql/postgresql_idx.py#L572

[anis016]

Typo fixes - While working on issue, please add a note

• https://github.com/ansible/ansible/blob/6a82827dddbe39d478bd148a66a44185ac52a399/lib/ansible/module_utils/facts/network/base.py#L44
• Find 'precendence' and change to 'precedence'
• Find 'arbitary' and change to 'arbitrary'
• Find 'sucessfully' and change to 'successfully'

merge request submitted: https://github.com/ansible/ansible/pull/52426

[Akasurde]

Undefined variable -

• https://github.com/ansible/ansible/blob/4a94bd8d9db2a4fd4a67097370e20dc8e5d90eeb/lib/ansible/modules/cloud/misc/rhevm.py#L593
• https://github.com/ansible/ansible/blob/4a94bd8d9db2a4fd4a67097370e20dc8e5d90eeb/lib/ansible/modules/files/archive.py#L410

[Akasurde]

low hanging fruits -
~https://github.com/ansible/ansible/issues/19463~
~https://github.com/ansible/ansible/issues/43318~

[Akasurde]

Typo -

• change from "registed" to "registered"
• change from "formating" to "formatting"
• change from "digeset" to "digest"

[Akasurde]

cc @userimack let me know if you are interested on working any of the above mentioned easyfixes.

[championshuttler]

Typo -

* change from "registed" to "registered"

* change from "formating" to "formatting"

* change from "digeset" to "digest"

Hi @Akasurde I would like to give a shot to them, can i take this up?

Thanks

[userimack]

@championshuttler yes, you can take it up, thanks

[ptoraskar]

Thank you @Akasurde! Really Appreciate!

[Akasurde]

Typo -

1. https://github.com/ansible/ansible/blob/2561fd700096ba83a6880c988fdcb89a91575235/lib/ansible/plugins/__init__.py#L78 - direclty to directly
2. https://github.com/ansible/ansible/blob/2561fd700096ba83a6880c988fdcb89a91575235/lib/ansible/plugins/__init__.py#L83 - standarize to standardize
3. https://github.com/ansible/ansible/blob/718a591fb24bbfc14a2c51567cd159bcbe17e525/lib/ansible/plugins/callback/counter_enabled.py#L134 thereyet to there yet

[Akasurde]

Typo -

Change 'dicionaries' to 'dictionaries' overall repo

[bhavikbhavsar]

Thanks @Akasurde

[Akasurde]

Typo fix - 'Mutiple' to 'Multiple'

[sbulage]

@Akasurde PR --> ~https://github.com/ansible/ansible/pull/64543~

[Akasurde]

PR ~https://github.com/ansible/ansible/pull/44375~ for ~https://github.com/ansible/ansible/issues/44340~ is about to close, needs love and care.

[sbulage]

@Akasurde Working on it.

[sbulage]

@Akasurde PR --> ~https://github.com/ansible/ansible/pull/64594~

[sbulage]

@valaparthvi

[Akasurde]

Help required in extending document - ~https://github.com/ansible/ansible/issues/59125~

[Akasurde]

Help required - ~https://github.com/ansible/ansible/issues/64628~

[sbulage]

@Akasurde working on it.

[Akasurde]

~https://github.com/ansible/ansible/issues/65223~

[sbulage]

@Akasurde working on it.

[sbulage]

@Akasurde PR --> ~https://github.com/ansible/ansible/pull/65229~

[Akasurde]

Help required - ~https://github.com/ansible/ansible/issues/40219~

[sbulage]

On it

[Akasurde]

~https://github.com/ansible/ansible/issues/62362~

[Akasurde]

~https://github.com/ansible/ansible/issues/65904~

[Akasurde]

~https://github.com/ansible/ansible/issues/66038~

[harshblog150]

Thanks @Akasurde

[Akasurde]

For new people - https://github.com/ansible/ansible/issues?q=is%3Aopen+is%3Aissue+label%3Aeasyfix+label%3Awaiting_on_contributor

[bmillemathias]

Help required in extending document - ansible/ansible#59125

I'm using this bug as an example on easyfix labeled bug.

Just adding easyfix label can helps spotting what are for fixes for developers that can be quick or simple from their point-of-view but for someone without any experience it is not easy, for a ton of reason. Perhaps because she does not know well python or the internals of the projects, or because she does not know how to check her code is correct.

Somethings can be automatically done, the bot can spot bug with easyfix label and add link to a page that help starting the project

• how to check out the code
• install the tools
• ...

We can also direct her to a specific room on IRC or distribution-list for supporting her.
But more than that I expect mentor and developer tagging easyfix to provide some guidance the moment they tagged easyfix with few word:

• the general idea to fix the bug or to find information
• the place where to fix in the repo.

Back to the example, for me this is not an easifix, because there is no direction to help the person to fix the problem

"regex can be configured to perform other tests as well".

I would expect, that the person put a comment with the other tests that could be done and possible a source (like the integration code) where there is examples.

Hopes it helps

[bmillemathias]

... (continuing)

Perhaps there is a misunderstand on the word easyfix because easy is quite a relative word. (we discuss about that on #ansible-devel). The text of the label easyfix is "The issue is considered easy to fix by aspiring contributors" seems subject to improvement.

I'd replace easyfix label by new_contributor and the criteria to be:

• code that can be fixed without a lot of knowledge on the internals (even the word lot is subject to discussion). Module are a good examples.
• impact of the code is quite low.
• integration test / unit code are fine I'd say
• is time a criteria ?

[imjoseangel]

But more than that I expect mentor and developer tagging easyfix to provide some guidance the moment they tagged easyfix with few word:

I really like the idea of the new_contributor label and most doing mentoring for new contributors. Personally I think some of us wouldn't mind to reserve some time to guide new contributors using IRC and using some examples. I try to do with people near me but I think having a "mentoring" community will open the door of new ideas and opportunities.

Thanks for your comments @bmillemathias

[imjoseangel]

This is a really good initiative for beginners. It has many forks and contributions. Just added Ansible as a reference https://github.com/firstcontributions/first-contributions

https://firstcontributions.github.io/

[bmillemathias]

I'd replace easyfix label by new_contributor and the criteria to be:

actually there is already a tag new_contributor given to people doing their first PR,

[imjoseangel]

Question. Is there any procedure or meeting to request and label an issue to be easyfix?

Thank you

[Akasurde]

Help required - https://github.com/ansible/awx/issues/924

[Akasurde]

Help required - ~https://github.com/ansible-collections/community.general/issues/529~

[felixfontein]

Help required - ansible-collections/community.general#1537

[Akasurde]

Help required -

• https://github.com/ansible-collections/community.general/issues/674
• https://github.com/ansible-collections/community.general/issues/635

[Akasurde]

Help required -

• https://github.com/ansible/ansible/issues/73262