[Mailman-Developers] Re: Cannnot find the processing of bounces as `hard` and `soft`

2019-07-20 Thread Mark Sapiro
On 7/20/19 11:45 AM, Aaryan Bhagat wrote:
> I understand, thanks for the reply, Mark.
> So that means I can set the `bounce_score_threshold` as Integer instead of 
> Float reason being that the score will always be increased by one.
> 
> Is the above inference correct?
> Am I missing something here?

It mm 2.1, it has always been a float to allow for the possible
implementation of fractional scores. It is still possible that
fractional scores will be implemented at some point in the future so my
inclination is that all these scores should be floats.

However, there is currently a contradiction in that

model/mailinglist.py has
bounce_score_threshold = Column(Integer) # XXX

but

styles/base.py has
mlist.bounce_score_threshold = 5.0

Whether we choose float or integer, we should be consistent in how we
define and set it.

Also, bounce_score_threshold is apparently not yet exposed in REST at all.

-- 
Mark Sapiro The highway is for gamblers,
San Francisco Bay Area, Californiabetter use your sense - B. Dylan
___
Mailman-Developers mailing list -- mailman-developers@python.org
To unsubscribe send an email to mailman-developers-le...@python.org
https://mail.python.org/mailman3/lists/mailman-developers.python.org/
Mailman FAQ: https://wiki.list.org/x/AgA3

Security Policy: https://wiki.list.org/x/QIA9


[Mailman-Developers] Re: flufl.lock._lockfile.NotLockedError: Already unlocked

2019-07-20 Thread Aaryan Bhagat
>This seems to be because you removed the var/ directory without stopping
>Mailman Core. Note that var/ directory is continuously needed by Mailman
>Core when running since it stores it's runtime information there. If you
>remove the var/ directory without stopping Mailman Core, you'll end
>up in weird situations like this.
>
>Make sure you always do mailman stop and only then remove the
>var/ directory.

Ok thanks, I understood it!
___
Mailman-Developers mailing list -- mailman-developers@python.org
To unsubscribe send an email to mailman-developers-le...@python.org
https://mail.python.org/mailman3/lists/mailman-developers.python.org/
Mailman FAQ: https://wiki.list.org/x/AgA3

Security Policy: https://wiki.list.org/x/QIA9


[Mailman-Developers] Re: Cannnot find the processing of bounces as `hard` and `soft`

2019-07-20 Thread Aaryan Bhagat
I understand, thanks for the reply, Mark.
So that means I can set the `bounce_score_threshold` as Integer instead of 
Float reason being that the score will always be increased by one.

Is the above inference correct?
Am I missing something here?
___
Mailman-Developers mailing list -- mailman-developers@python.org
To unsubscribe send an email to mailman-developers-le...@python.org
https://mail.python.org/mailman3/lists/mailman-developers.python.org/
Mailman FAQ: https://wiki.list.org/x/AgA3

Security Policy: https://wiki.list.org/x/QIA9


[Mailman-Developers] Re: flufl.lock._lockfile.NotLockedError: Already unlocked

2019-07-20 Thread Abhilash Raj



On Sat, Jul 20, 2019, at 7:15 AM, Aaryan Bhagat wrote:
> I have started working on 
> [this](https://gitlab.com/mailman/mailman/merge_requests/538) pr. It is 
> regarding various bounce_process functions. Strange behaviour is 
> observed which I am not so sure what or why is happening. Unexpected 
> Traceback error pops up in my terminal.
> 
> >File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 580, in 
> >main
> >loop.loop()
> >  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 401, 
> > in loop
> >self._pause()
> >  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 391, 
> > in _pause
> >signal.pause()
> >  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 267, 
> > in sigalrm_handler
> >self._lock.refresh()
> >  File 
> > "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/flufl.lock-3.2-py3.7.egg/flufl/lock/_lockfile.py",
> >  line 184, in refresh
> >flufl.lock._lockfile.NotLockedError:  >/home/aaryan/mailman/mailman/var/locks/master.lck [unlocked: 1 day, 6:00:00] 
> >pid=7596 at 0x7f54af802320x>: None
> >
> >During handling of the above exception, another exception occurred:
> >
> >Traceback (most recent call last):
> >  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 583, 
> > in main
> >os.remove(config.PID_FILE)
> >FileNotFoundError: [Errno 2] No such file or directory: 
> >'/home/aaryan/mailman/mailman/var/master.pid'
> >
> >During handling of the above exception, another exception occurred:
> >
> >Traceback (most recent call last):
> >  File "/home/aaryan/mailman/venv3/bin/master", line 11, in 
> >load_entry_point('mailman', 'console_scripts', 'master')()
> >  File 
> > "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/Click-7.0-py3.7.egg/click/core.py",
> >  line 764, in __call__
> >return self.main(*args, **kwargs)
> >  File 
> > "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/Click-7.0-py3.7.egg/click/core.py",
> >  line 717, in main
> >rv = self.invoke(ctx)
> >  File 
> > "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/Click-7.0-py3.7.egg/click/core.py",
> >  line 956, in invoke
> >return ctx.invoke(self.callback, **ctx.params)
> >  File 
> > "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/Click-7.0-py3.7.egg/click/core.py",
> >  line 555, in invoke
> >return callback(*args, **kwargs)
> >  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 585, 
> > in main
> >lock.unlock()
> >  File 
> > "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/flufl.lock-3.2-py3.7.egg/flufl/lock/_lockfile.py",
> >  line 287, in unlock
> >flufl.lock._lockfile.NotLockedError: Already unlocked
> 
> I think some configuration of mailman is conflicting. One point to 
> mention is the `var` was created when I was working on some other 
> branch.

This seems to be because you removed the var/ directory without stopping
Mailman Core. Note that var/ directory is continuously needed by Mailman
Core when running since it stores it's runtime information there. If you
remove the var/ directory without stopping Mailman Core, you'll end
up in weird situations like this.

Make sure you always do `mailman stop` and only then remove the
var/ directory.

> Maybe deleting the var and running `mailman start` will solve it?
> Not many leads are there hence I though best asking this.
> Pointers to this problem are appreciated
> Am I missing something here?
> ___
> Mailman-Developers mailing list -- mailman-developers@python.org
> To unsubscribe send an email to mailman-developers-le...@python.org
> https://mail.python.org/mailman3/lists/mailman-developers.python.org/
> Mailman FAQ: https://wiki.list.org/x/AgA3
> 
> Security Policy: https://wiki.list.org/x/QIA9
>

-- 
  thanks,
  Abhilash Raj (maxking)
___
Mailman-Developers mailing list -- mailman-developers@python.org
To unsubscribe send an email to mailman-developers-le...@python.org
https://mail.python.org/mailman3/lists/mailman-developers.python.org/
Mailman FAQ: https://wiki.list.org/x/AgA3

Security Policy: https://wiki.list.org/x/QIA9


[Mailman-Developers] Re: Cannnot find the processing of bounces as `hard` and `soft`

2019-07-20 Thread Mark Sapiro
On 7/19/19 10:50 PM, Aaryan Bhagat wrote:
> 
> Currently, in the `mailman/src/mailman/runners/bounce.py` I do not see 
> whether the separation of bounces as `hard` and `soft` is implemented. 
> According to the previous version of Mailman, the `bounce_score` is increased 
> accordingly ( see the `Bounce processing` section of 
> [this](https://wiki.list.org/DOC/Mailman%202.1%20List%20Administrators%20Manual)
>  page ) depending upon the type of bounce. I intend to follow that 
> implementation. If not required I can drop that implementation and just 
> increase the `bounce_score` as an integer always.


Contrary to the documentation, Mailman has never distinguished between
"hard" and "soft" bounces. The recognizer used in Mailman 3,
flufl.bounce, implements essentially the same process as Mailman 2.1. It
distinguishes between permanent (5xx) failures and temporary (4xx)
failures, but not between various kinds of permanent failures, e.g.
unknown user vs full mailbox.

Mailman 2.1 has never actually scored "soft" bounces as 0.5. It scores
5xx bounces as 1.0 and ignores 4xx bounces.

The same should be done with Mailman 3. Distinguishing between "hard"
and "soft" 5xx bounces would require significant changes to flufl.bounce
and is outside the scope of your project.

-- 
Mark Sapiro The highway is for gamblers,
San Francisco Bay Area, Californiabetter use your sense - B. Dylan
___
Mailman-Developers mailing list -- mailman-developers@python.org
To unsubscribe send an email to mailman-developers-le...@python.org
https://mail.python.org/mailman3/lists/mailman-developers.python.org/
Mailman FAQ: https://wiki.list.org/x/AgA3

Security Policy: https://wiki.list.org/x/QIA9


[Mailman-Developers] Re: Cannnot find the processing of bounces as `hard` and `soft`

2019-07-20 Thread Abhilash Raj
On Fri, Jul 19, 2019, at 10:51 PM, Aaryan Bhagat wrote:
> Somewhat details regarding this can be found in 
> [this](https://mail.python.org/archives/list/mailman-developers@python.org/thread/N7LB2VQN5MC7INXTP3N2CF7MJYSZQI3W/#5ELFHHN4FDLXHXLDVBYKLX36GQTWJTWV)
>  thread also.
> 
> Currently, in the `mailman/src/mailman/runners/bounce.py` I do not see 
> whether the separation of bounces as `hard` and `soft` is implemented. 

Looking at the code in `src/runners/bounce.py` I don't think temporary
failures are even stored. It seems like they are just ignored. 

> According to the previous version of Mailman, the `bounce_score` is 
> increased accordingly ( see the `Bounce processing` section of 
> [this](https://wiki.list.org/DOC/Mailman%202.1%20List%20Administrators%20Manual)
>  page ) depending upon the type of bounce. I intend to follow that 
> implementation. If not required I can drop that implementation and just 
> increase the `bounce_score` as an integer always.

I don't think we should be increasing the bounce_score on soft failures, which 
can be retried by the MTA. 

> 
> Pointers on this issue are appreciated.
> Am I missing something here?
> ___
> Mailman-Developers mailing list -- mailman-developers@python.org
> To unsubscribe send an email to mailman-developers-le...@python.org
> https://mail.python.org/mailman3/lists/mailman-developers.python.org/
> Mailman FAQ: https://wiki.list.org/x/AgA3
> 
> Security Policy: https://wiki.list.org/x/QIA9
>

-- 
  thanks,
  Abhilash Raj (maxking)
___
Mailman-Developers mailing list -- mailman-developers@python.org
To unsubscribe send an email to mailman-developers-le...@python.org
https://mail.python.org/mailman3/lists/mailman-developers.python.org/
Mailman FAQ: https://wiki.list.org/x/AgA3

Security Policy: https://wiki.list.org/x/QIA9


[Mailman-Developers] flufl.lock._lockfile.NotLockedError: Already unlocked

2019-07-20 Thread Aaryan Bhagat
I have started working on 
[this](https://gitlab.com/mailman/mailman/merge_requests/538) pr. It is 
regarding various bounce_process functions. Strange behaviour is observed which 
I am not so sure what or why is happening. Unexpected Traceback error pops up 
in my terminal.

>File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 580, in 
>main
>loop.loop()
>  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 401, in 
> loop
>self._pause()
>  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 391, in 
> _pause
>signal.pause()
>  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 267, in 
> sigalrm_handler
>self._lock.refresh()
>  File 
> "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/flufl.lock-3.2-py3.7.egg/flufl/lock/_lockfile.py",
>  line 184, in refresh
>flufl.lock._lockfile.NotLockedError: /home/aaryan/mailman/mailman/var/locks/master.lck [unlocked: 1 day, 6:00:00] 
>pid=7596 at 0x7f54af802320x>: None
>
>During handling of the above exception, another exception occurred:
>
>Traceback (most recent call last):
>  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 583, in 
> main
>os.remove(config.PID_FILE)
>FileNotFoundError: [Errno 2] No such file or directory: 
>'/home/aaryan/mailman/mailman/var/master.pid'
>
>During handling of the above exception, another exception occurred:
>
>Traceback (most recent call last):
>  File "/home/aaryan/mailman/venv3/bin/master", line 11, in 
>load_entry_point('mailman', 'console_scripts', 'master')()
>  File 
> "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/Click-7.0-py3.7.egg/click/core.py",
>  line 764, in __call__
>return self.main(*args, **kwargs)
>  File 
> "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/Click-7.0-py3.7.egg/click/core.py",
>  line 717, in main
>rv = self.invoke(ctx)
>  File 
> "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/Click-7.0-py3.7.egg/click/core.py",
>  line 956, in invoke
>return ctx.invoke(self.callback, **ctx.params)
>  File 
> "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/Click-7.0-py3.7.egg/click/core.py",
>  line 555, in invoke
>return callback(*args, **kwargs)
>  File "/home/aaryan/mailman/mailman/src/mailman/bin/master.py", line 585, in 
> main
>lock.unlock()
>  File 
> "/home/aaryan/mailman/venv3/lib/python3.7/site-packages/flufl.lock-3.2-py3.7.egg/flufl/lock/_lockfile.py",
>  line 287, in unlock
>flufl.lock._lockfile.NotLockedError: Already unlocked

I think some configuration of mailman is conflicting. One point to mention is 
the `var` was created when I was working on some other branch.
Maybe deleting the var and running `mailman start` will solve it?
Not many leads are there hence I though best asking this.
Pointers to this problem are appreciated
Am I missing something here?
___
Mailman-Developers mailing list -- mailman-developers@python.org
To unsubscribe send an email to mailman-developers-le...@python.org
https://mail.python.org/mailman3/lists/mailman-developers.python.org/
Mailman FAQ: https://wiki.list.org/x/AgA3

Security Policy: https://wiki.list.org/x/QIA9